Skip to content

Fix #17553 - [Metro] Unreadable links and UI fixes/improvements#17665

Merged
williamdes merged 8 commits intophpmyadmin:QA_5_2from
falconandrea:fix-issue-17553
Oct 29, 2022
Merged

Fix #17553 - [Metro] Unreadable links and UI fixes/improvements#17665
williamdes merged 8 commits intophpmyadmin:QA_5_2from
falconandrea:fix-issue-17553

Conversation

@falconandrea
Copy link
Copy Markdown
Contributor

Description

Fix bugs reported in issue #17553.

I made some additional CSS fix

  • changed background-color to the alert box in /server/databases route for better readability
  • removed button styles from the print link in database pages and lock link in user accounts page for unify styles with other links
  • changed hover styles on buttons, for example Drop button in /server/databases route (where I changed also the icon with the white color version)

I hope it is all correct. This is my first Pull Request on OpenSource World :)

Signed-off-by: Andrea Falcon <falcon.andrea88@gmail.com>
Signed-off-by: Andrea Falcon <falcon.andrea88@gmail.com>
Signed-off-by: Andrea Falcon <falcon.andrea88@gmail.com>
Signed-off-by: Andrea Falcon <falcon.andrea88@gmail.com>
Signed-off-by: Andrea Falcon <falcon.andrea88@gmail.com>
Signed-off-by: Andrea Falcon <falcon.andrea88@gmail.com>
Signed-off-by: Andrea Falcon <falcon.andrea88@gmail.com>
Signed-off-by: Andrea Falcon <falcon.andrea88@gmail.com>
@williamdes williamdes requested a review from MauricioFauth July 29, 2022 09:01
@codecov

This comment was marked as off-topic.

@williamdes
Copy link
Copy Markdown
Member

Hi @MauricioFauth
could you have a look to this one ?

@williamdes williamdes changed the title Fix issue 17553 Fix #17553 - [Metro] Unreadable links and close button Oct 29, 2022
@williamdes williamdes linked an issue Oct 29, 2022 that may be closed by this pull request
@williamdes williamdes self-assigned this Oct 29, 2022
@williamdes williamdes removed the request for review from MauricioFauth October 29, 2022 17:16
@williamdes williamdes added this to the 5.2.1 milestone Oct 29, 2022
@williamdes williamdes added Enhancement A feature request for improving phpMyAdmin UI Issues relating to the user interface labels Oct 29, 2022
Copy link
Copy Markdown
Member

@williamdes williamdes left a comment

Choose a reason for hiding this comment

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

Again I want to say a big thank you for this pull-request !
It's very good work for a first time in open-source !! 🎉

It took way too long for me to review this one in depth, sorry for that :/

The points I reviewed:

changed background-color to the alert box in /server/databases route for better readability

Perfect !

removed button styles from the print link in database pages and lock link in user accounts page for unify styles with other links

Very good !

changed hover styles on buttons, for example Drop button in /server/databases route (where I changed also the icon with the white color version)

Way better in white !

Good work on the login page too !

I reverted the change made for the modal close button as it is worse now that fixes where made for it as I said on #17553

@williamdes williamdes changed the title Fix #17553 - [Metro] Unreadable links and close button Fix #17553 - [Metro] Unreadable links and UI fixes/improvements Oct 29, 2022
@williamdes williamdes merged commit 3ade69a into phpmyadmin:QA_5_2 Oct 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement A feature request for improving phpMyAdmin UI Issues relating to the user interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Metro] Unreadable button links

2 participants