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

Change styles on "Read more" button on RFC card #399

Merged
merged 1 commit into from
Nov 17, 2023

Conversation

SerhiiCho
Copy link
Collaborator

Style changes of the "Read more" button on RFC card:

Light theme

image

Dark theme

image

It has a nice hover effect, which is not visible in the screenshot.

This PR is related to #391 issue where @theofidry mentioned that:

When checking another RFCs, currently you see the small box with "Read more" and need to click on the "Read more" link to get there. Maybe the whole thumbnail could be a link instead?

As a solution, I think this small change will make it easier to go to an RFC page. Make the whole card as a link is not really possible because we'll need to remove all the links from the inside of the card, since in HTML, you can't have a link inside a link. There is a workaround using JavaScript and click event and window.location.href = 'some link', but I don't think I want to do that.

@SerhiiCho SerhiiCho added the ux/ui Issues and PRs related to UX/UI design label Nov 17, 2023
@SerhiiCho SerhiiCho self-assigned this Nov 17, 2023
@SerhiiCho SerhiiCho merged commit 042a973 into brendt:main Nov 17, 2023
3 checks passed
@SerhiiCho SerhiiCho deleted the style-changes-read-more-button branch November 17, 2023 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ux/ui Issues and PRs related to UX/UI design
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant