<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 19, 2015 at 3:18 PM, Kushagra Pandey <span dir="ltr"><<a href="mailto:kushagra4296@gmail.com" target="_blank">kushagra4296@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Thu, Mar 19, 2015 at 7:55 AM, Madhura Jayaratne <span dir="ltr"><<a href="mailto:madhura.cj@gmail.com" target="_blank">madhura.cj@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div>On Wed, Mar 18, 2015 at 10:56 PM, Kushagra Pandey <span dir="ltr"><<a href="mailto:kushagra4296@gmail.com" target="_blank">kushagra4296@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Hi Issac<br><div class="gmail_extra"><br><div class="gmail_quote"><div><div>On Wed, Mar 18, 2015 at 7:31 PM, Isaac Bennetch <span dir="ltr"><<a href="mailto:bennetch@gmail.com" target="_blank">bennetch@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div><br>
<br>
On 3/18/15 9:58 AM, Isaac Bennetch wrote:<br>
> Hi Madhura,<br>
><br>
> On 3/18/15 6:26 AM, Madhura Jayaratne wrote:<br>
>><br>
>><br>
>> On Fri, Mar 13, 2015 at 3:03 PM, Madhura Jayaratne <<a href="mailto:madhura.cj@gmail.com" target="_blank">madhura.cj@gmail.com</a><br>
>> <mailto:<a href="mailto:madhura.cj@gmail.com" target="_blank">madhura.cj@gmail.com</a>>> wrote:<br>
>><br>
>>     Hi all,<br>
>><br>
>>     As per the discussion on the recent team meeting [1] I have started<br>
>>     working on updating metro theme to be compatible with phpMyAdmin 4.4<br>
>>     and you can find what I've done so far at [2]. I am basically going<br>
>>     through all the commits that had some style/image change since 2013<br>
>>     and adapting them to metro them to make sure nothing is missed.<br>
>><br>
>>     [1] <a href="https://wiki.phpmyadmin.net/pma/2015-03_Meeting#Themes" target="_blank">https://wiki.phpmyadmin.net/pma/2015-03_Meeting#Themes</a><br>
>>     [2] <a href="https://github.com/madhuracj/metro/commits/master" target="_blank">https://github.com/madhuracj/metro/commits/master</a><br>
>><br>
>>     --<br>
>>     Thanks and Regards,<br>
>><br>
>>     Madhura Jayaratne<br>
>><br>
>><br>
>> Hi devs,<br>
>><br>
>> I think the theme is in a reasonable state now. Would appreciate if you<br>
>> could get the theme from [1] and try it with phpMyAdmin version 4.4 (in<br>
>> Beta) and give feedback.<br>
>><br>
>> [1] <a href="https://github.com/madhuracj/metro/" target="_blank">https://github.com/madhuracj/metro/</a><br>
><br>
> Great progress! Very nice job. I have some thoughts:<br>
><br>
> * On the login page, there's a large 'a' displayed. I don't remember<br>
> much about the metro theme from times past, so maybe the original<br>
> designer had some motivation for it, but I'd rather see a p or our logo<br>
> or something there.<br></div></div></blockquote></div></div><div>Seems somehow IcoMoon font didn't load on your system. Kindly refresh. It's a user icon there. </div><span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div>
><br>
> * On the main page, I see something extra after the Theme selection<br>
> dropdown: "Scheme: win" do you have thoughts about that? I do not see it<br>
> with pmahomme<br>
><br>
> * From a table structure page, under the list of columns there's the<br>
> list of actions; Print view, Propose table structure, etc. The spacing<br>
> between the first three (Print view, Propose table structure, and Track<br>
> table) look okay, but then the rest seem to be squished closer together.<br>
> This is particularly noticeable because the 'e' from "Track table" seems<br>
> to be slightly covered by the icon for "Move columns"<br>
><br>
> * The green color used, for instance, when showing a successful query<br>
> "Showing rows 0 - 8 (9 total, Query took 0.0009 seconds.)" is a bit<br>
> bright for the tone of the rest of the theme. It's currently #01A31C,<br>
> perhaps something like #77BB77 would fit better.<br>
><br>
> * The navigation icons, Home, Log out, phpMyAdmin documentation, MySQL<br>
> documentation, and Reload navigation frame -- none of the icons are<br>
> appearing for me. For Home I see an exclamation point, and the rest of<br>
> them have the appearance of unsupported unicode characters. Is this a<br>
> problem with my Firefox?<br></div></div></blockquote></span><div>No such issues on my Chrome. </div><span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div><div>
><br>
> Again, great work.<br>
> ~isaac<br>
><br>
<br>
</div></div>Also, for some reason I don't see a tool tip for the MySQL documentation<br>
when I hover over that icon, but I do see it with the other two themes.<br></blockquote></span><div>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? </div></div></div></div></blockquote><div><br></div></div></div><div>Hi Kushagra,</div><div><br></div><div>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.</div></div><span><br clear="all"></span></div></div></blockquote></div></div><div>Hi Madhura, I have submitted a pull request here: <a href="https://github.com/phpmyadmin/phpmyadmin/pull/1595" target="_blank">https://github.com/phpmyadmin/phpmyadmin/pull/1595</a></div><div>Let me know if something needs to be changed. I changed "documentation" to "MySQL Documentation" for clarity. </div><div>I have been little occupied with trying something to do (bug fix/feature implementation) for GSOC - so once I get free, I will contribute to the theme, as I guess contribution to theme will not be counted as one for GSOC.</div><div><br></div></div></div></div></blockquote><div>AFAIK, it is not counted unless your GSoC project is in the same area. Others can verify this. </div></div><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Thanks and Regards,<div><br></div><div>Madhura Jayaratne<br><div><br></div></div></div>
</div></div>