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
Typography user show #8140
Typography user show #8140
Conversation
We applied some changes regarding the contrast in the username. Apart from that, we grouped the email with the other elements at the bottom.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest leaving the email where it was. It's part of a user's details so it should be with the name, login and roles. It is also pushed down if a user is part of many groups.
@hennevogel @dmarcoux @DavidKang An alternative for the user's info area could be: I simplified the beta stuff using only one switch instead of a link and a badget. I added a new commit that can be squashed after review. |
e213a49
to
72792e7
Compare
@hennevogel, what about this approach, regarding the "beta program" |
@DavidKang awesome |
56bd7b5
to
5a39f08
Compare
Review app will appear here: http://obs-reviewlab.opensuse.org/davidkang-typography-user-show |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The switch isn't working like we defined in our patterns.
fe8597b
to
1e3ddd1
Compare
= check_box_tag('user[in_beta]', !user.in_beta, user.in_beta, class: 'custom-control-input', id: 'beta-switch') | ||
= label_tag 'Public Beta Program', nil, class: 'custom-control-label', for: 'beta-switch' | ||
= hidden_field_tag('user[login]', user.login) | ||
%i.fa.fa-question-circle.text-info{ data: { placement: 'top', toggle: 'popover', html: 'true', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
%i.fa.fa-question-circle.text-info{ data: { placement: 'top', toggle: 'popover', html: 'true', | |
%i.fa.fa-question-circle.text-info{ data: { placement: 'top', toggle: 'popover', html: 'true', | |
content: 'In our <strong>beta program</strong> you can try ' + | |
'the latest features we develop, so you can give us feedback before we release them.' } } |
Co-authored-by: David Kang <dkang@suse.com> Co-authored-by: Lukas Krause <lkrause@suse.de>
1e3ddd1
to
ea59658
Compare
It was approved except for the text, which is already updated
We applied some changes regarding the contrast in the username. Apart from that, we grouped the email with the other elements at the bottom.
Before
After
Beta Program Tooltip: