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

Frontend styling refactor / UI polish #478

Merged
merged 26 commits into from
Jun 20, 2022
Merged

Frontend styling refactor / UI polish #478

merged 26 commits into from
Jun 20, 2022

Conversation

kristoferlund
Copy link
Member

I have started polishing up the UI. There is still lots to do, mostly in terms of responsiveness and dark mode. But I think this is bug free enough to merge for now.

  • Added: A username popover everywhere there is a username.

Skärmavbild 2022-06-17 kl  08 42 01

Skärmavbild 2022-06-17 kl  08 40 33

Skärmavbild 2022-06-17 kl  08 44 03

@kristoferlund kristoferlund requested review from mattyg and nebs-dev June 17, 2022 19:12
Copy link
Collaborator

@mattyg mattyg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

man incredible work. It's so beautiful!!

One other idea I didn't mention inline: Could we maybe add a "secondary" button style? That way if we have two buttons side-by-side (like in the QuantifyTable), we could visusally distinguish them somewhat.

@kristoferlund
Copy link
Member Author

man incredible work. It's so beautiful!!

One other idea I didn't mention inline: Could we maybe add a "secondary" button style? That way if we have two buttons side-by-side (like in the QuantifyTable), we could visusally distinguish them somewhat.

One common way to deal with this is to use an "outline" button style with the same colour. That is then used for with "less important" buttons, in relation to some "more important" button.

In the case of the quantify screen, both buttons are of equal importance which makes it more difficult to motivate why they should have different colours.

@kristoferlund kristoferlund merged commit 516668b into main Jun 20, 2022
@kristoferlund kristoferlund deleted the fix/praise_layout branch June 20, 2022 12:45
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.

2 participants