On Wed, Mar 18, 2015 at 10:56 PM, Kushagra Pandey kushagra4296@gmail.com wrote:
Hi Issac
On Wed, Mar 18, 2015 at 7:31 PM, Isaac Bennetch bennetch@gmail.com wrote:
On 3/18/15 9:58 AM, Isaac Bennetch wrote:
Hi Madhura,
On 3/18/15 6:26 AM, Madhura Jayaratne wrote:
On Fri, Mar 13, 2015 at 3:03 PM, Madhura Jayaratne <
madhura.cj@gmail.com
mailto:madhura.cj@gmail.com> wrote:
Hi all, As per the discussion on the recent team meeting [1] I have started working on updating metro theme to be compatible with phpMyAdmin
4.4
and you can find what I've done so far at [2]. I am basically going through all the commits that had some style/image change since 2013 and adapting them to metro them to make sure nothing is missed. [1] https://wiki.phpmyadmin.net/pma/2015-03_Meeting#Themes [2] https://github.com/madhuracj/metro/commits/master -- Thanks and Regards, Madhura Jayaratne
Hi devs,
I think the theme is in a reasonable state now. Would appreciate if you could get the theme from [1] and try it with phpMyAdmin version 4.4 (in Beta) and give feedback.
Great progress! Very nice job. I have some thoughts:
- On the login page, there's a large 'a' displayed. I don't remember
much about the metro theme from times past, so maybe the original designer had some motivation for it, but I'd rather see a p or our logo or something there.
Seems somehow IcoMoon font didn't load on your system. Kindly refresh. It's a user icon there.
- On the main page, I see something extra after the Theme selection
dropdown: "Scheme: win" do you have thoughts about that? I do not see it with pmahomme
- From a table structure page, under the list of columns there's the
list of actions; Print view, Propose table structure, etc. The spacing between the first three (Print view, Propose table structure, and Track table) look okay, but then the rest seem to be squished closer together. This is particularly noticeable because the 'e' from "Track table" seems to be slightly covered by the icon for "Move columns"
- The green color used, for instance, when showing a successful query
"Showing rows 0 - 8 (9 total, Query took 0.0009 seconds.)" is a bit bright for the tone of the rest of the theme. It's currently #01A31C, perhaps something like #77BB77 would fit better.
- The navigation icons, Home, Log out, phpMyAdmin documentation, MySQL
documentation, and Reload navigation frame -- none of the icons are appearing for me. For Home I see an exclamation point, and the rest of them have the appearance of unsupported unicode characters. Is this a problem with my Firefox?
No such issues on my Chrome.
Again, great work. ~isaac
Also, for some reason I don't see a tool tip for the MySQL documentation when I hover over that icon, but I do see it with the other two themes.
Well, other icons <a> tag have a title attribute, but the MySQL documentation <a> is missing the title attribute. Other themes are showing the tooltips on the <img> tag (child of <a> tag), but this theme is showing the <a>'s title attribs. Should I add the 'title' attrib to this <a> in phpmyadmin code?
Hi Kushagra,
You may submit a pull request against QA_4_4 branch. Also, since the layout seems to be ok now, you are most welcome to work on improving the theme.