-
-
Notifications
You must be signed in to change notification settings - Fork 56
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
More conspicuous download button #1094
Conversation
4e439a2
to
ded7e29
Compare
@kelson42 Who is going to review this PR? I wanted to add you as a reviewer, but GitHub won't let me do it (likely, because you are the "author" of this PR). |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1094 +/- ##
=======================================
Coverage 41.43% 41.43%
=======================================
Files 59 59
Lines 4245 4245
Branches 2323 2323
=======================================
Hits 1759 1759
Misses 990 990
Partials 1496 1496 ☔ View full report in Codecov by Sentry. |
ded7e29
to
f9b9f64
Compare
@veloman-yunkan Please review and fix. The code is from @juuz0. I think the proposal of improvement is OK. I have only one small concern: the grey borders around the new blue button even if perfectly there, look inexistant which - at least to me - give the impression that the button is smaller than the box (1px left and 1px right). Do you have also this feeling? |
I know. I already modified @juuz0's changes, pushed the branch and was going to create a PR when GitHub told me that there is already a PR (this one) associated with that branch. It would be strange if I review my own PR.
Which proposal of improvement do you mean?
It didn't catch my attention but after looking closely - yes, it looks like that. |
31dc5b4
to
b3f0d2e
Compare
Fixed by changing the border color to that of the button itself. |
Fixed in a different way. @kelson42 Please test again. |
@kelson42 Note that I didn't update the cacheid's. I will update them (and make the CI pass) while squashing the fixup commits once we are done with the visual appearance issues. |
@veloman-yunkan Thx! Good to merge then when commits are reorganised! |
- Changed the position of download button to the end of tile and added a proper download icon to it. When the button is hovered it becomes darker. - Also internationalized the "Download" text on the modal download widget and added download size information to it.
4b1c201
to
c3d94cd
Compare
Done |
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.
No-JS page contains a usability bug (#1130).
@veloman-yunkan No absolutly sure about your approach/opinion, but to me #1130 should not be introduced by this PR (so fixed here). |
... to keep it in sync with the JSful library page that has been modified in the previous commit.
c3d94cd
to
3cdc036
Compare
Done |
Fixes #1040
Fixes #1130