-
Notifications
You must be signed in to change notification settings - Fork 152
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
Notification numbers are not properly aligned #398
Comments
I would love to work on this |
Awesome, thanks @RabsonJ 👍 |
Thank you! Let me get working |
@hanzei can you please give me the exact name of the file / it's location |
Sure @RabsonJ. The button are defined in https://github.com/mattermost/mattermost-plugin-github/blob/master/webapp/src/components/sidebar_buttons/sidebar_buttons.jsx. You might want to take a look at the mattermost-plugin-github/webapp/src/components/sidebar_buttons/sidebar_buttons.jsx Lines 211 to 235 in 793631a
|
I would Like to work on this issue . |
Go ahead, @JanumalaAkhilendra, all yours! |
The numbers in the left hand side bar are not properly aligned, which is very obvious when all the numbers are the same ones, as in the following screenshot:
We need to fix the CSS so they are aligned. I'm not sure if the best way would be to assign the same width to the icons, or to align the icons and the numbers separately.
The text was updated successfully, but these errors were encountered: