Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed UI for Proposal List Display and UI for links in full proposal #854

Merged
merged 3 commits into from
Jun 14, 2024

Conversation

ashutoshpw
Copy link
Collaborator

@ashutoshpw ashutoshpw commented Jun 14, 2024

This PR has two minor updates

  • Content is trimmed in the proposal list if the length of the text content is more than 300 characters
  • Color of links within the proposal details page is changed to light blue for better visibility.
Before After
Screenshot 2024-06-14 at 5 26 09 PM Screenshot 2024-06-14 at 5 24 43 PM
image Screenshot 2024-06-14 at 5 28 01 PM

Copy link

netlify bot commented Jun 14, 2024

Deploy Preview for tezos-homebase ready!

Name Link
🔨 Latest commit 7adfc84
🔍 Latest deploy log https://app.netlify.com/sites/tezos-homebase/deploys/666c35a42c8d9600073beb8f
😎 Deploy Preview https://deploy-preview-854--tezos-homebase.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@csoreff
Copy link
Collaborator

csoreff commented Jun 14, 2024

Let's use the same color for links + underline to match the on-chain proposal pages:

image

@ashutoshpw
Copy link
Collaborator Author

Updated the link colors on the proposal details page. However, this change will not take effect if the proposal HTML already specifies a link color within the style attribute.

@csoreff csoreff merged commit 794f391 into master Jun 14, 2024
6 checks passed
@csoreff csoreff deleted the chore/proposal-ui branch June 14, 2024 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants