-
-
Notifications
You must be signed in to change notification settings - Fork 647
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
fix article control buttons #731
fix article control buttons #731
Conversation
I like this. Can we make all buttons like this? Also pin and delete. I also think we need different icons for publish and decline. Maybe these two? Approve: https://blade-ui-kit.com/blade-icons/heroicon-s-check |
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 really like this too, great job. Just a couple of minor formatting tweaks from me.
@driesvints I'm happy with this when you are |
Thanks @faissaloux! |
I just noticed this PR causes a weird issue with buttons expanding on the bottom padding when you hover over them. Can we get that fixed? |
Yes @driesvints I just noticed the same issue I'll fix it |
@driesvints #732 fixes this height issue |
@faissaloux I just noticed that I can't seem to click any of the buttons anymore in Safari after this. |
@driesvints fixed here #747 |
resolves #728
2021-11-09.00-25-36.mp4