Jump to content

New ProcessWire admin theme on dev branch


ryan

Recommended Posts

To me it looks like faux bold.. I wasn't looking into it so sorry if that turns out to be the problem and that's a common problem when people use custom fonts. Some browsers are worse than others, but when working with custom font make sure you choose a font that has at least 2-3 weights and set them according. Just making a font "bold" doesn't mean it has a "bold" (600?) weight, make sure that you use the weight defined by the web font mostly like 500-700.

Link to comment
Share on other sites

Craig, thanks for your tests. Both of your screenshots looked pretty decent for Windows. So it sounds like the one on the right is Arial? Arimo looks pretty good on the left (at least compared to the earlier Windows screenshots), but relative to Arial, it looks a rather fuzzy.

Yes, the right one was Arial, and the left one was Arimo with SVG at the top of the font "src" stack.

I've had a look at the Google Fonts page, and the result is shown in the attached image. It renders clearly, but again - it's the bold 'e' at that size that is the ugly duckling. For comparison, I added Open Sans.

post-1537-0-55095500-1383584556_thumb.pn

Link to comment
Share on other sites

Nice start! Already like it much better than the current theme.

Some things I noticed:

- Everything looks nicer if you limit the container to about 1050px in my opinion

- see setup/templates or setup/fields: the filter box would look nicer with some top margin and

- the style when you hover the icons in the table don't fit the design at all

Link to comment
Share on other sites

The new "Add New" button is nice feature but I'm not sure it's very useful for all sites (the way it works for now), if I have a site structure that doesn't allow for the right setup needed. Either because the template may used in different branches, and or a multisite setup.

I dream of a more configurable thing, and maybe also able to disable it.

  • Like 3
Link to comment
Share on other sites

I also would like to make it more configurable, and definitely plan to. But in the short term, I think the best bet is just have it not show if there aren't any add-ready templates to populate it with. I will do this before 2.4 is final, but just wanted to keep it there with a little instruction notice to introduce it, as I thought people might not realize the option was there otherwise. 



I've had a look at the Google Fonts page, and the result is shown in the attached image. It renders clearly, but again - it's the bold 'e' at that size that is the ugly duckling. For comparison, I added Open Sans.

Thanks for the screenshot comparison. It looks like the "e" also has the same issue with the bold Open Sans, though to a lesser extent. I notice the 12px size seems to look fine with both by comparison. Is the problem just the 14px font size with these fonts in Windows? Would using a different size (bigger or smaller) as a default be better in the Windows environment?

  • Like 1
Link to comment
Share on other sites

Even with those styles applied I don't see a real difference.

And as there are no css fixes for Firefox or IE it's not really a help for this Windows webfonts problem.

It's also not much better for the following Open Sans and Source Sans Pro:

post-1863-0-94314000-1384094634_thumb.pn

post-1863-0-48414500-1384094647_thumb.pn

  • Like 1
Link to comment
Share on other sites

Thanks for the screenshots Wayne. To my eyes, it looks like the 13px–15px range is problematic for Arimo and Windows, and the other fonts as well. With Arimo, both 12px and 16px look just fine to me (how about you guys?). Perhaps we can solve this by just using a smaller default size that corresponds with 12px. To me 16px just seems overly large for body readable text, like a large print book for visually impaired people. But it is typically the browser default, so not positive whether to default up or down on this one... learning towards down (12px), but going to experiment. 

Link to comment
Share on other sites

I want 15.245px :P

Just noticed that when I have images field in a custom tab as the only field (is at top), the grid view doesn't work. At least for me in latest version Chrome. ©licking the icon just collapses the field.

Link to comment
Share on other sites

Amen.. oh, I mean Arial.

yes, a lot of time spent to try other fonts... Wouldn't it be easier to stay with 'Arial, Helvetica, sans-serif'?

Another point (already mentioned earlier). In the setup menu dropdowns the translation isn't available. Only 'languages' is translated.

post-1027-0-79964700-1384191835_thumb.jp

Link to comment
Share on other sites

I want 15.245px  :P

Actually it looks like 15px (0.9375em in our case) is a good route. This seems to render nicely in the Arimo screenshots above, and looks about the same as 14px in OS X. The 16px size I tested just seems way too big, but 15px seems like a nice balance. Is there something special about 15.245 or are you just being silly? :) You know I'm actually going to go test it out now. :) 

Just noticed that when I have images field in a custom tab as the only field (is at top), the grid view doesn't work. At least for me in latest version Chrome. ©licking the icon just collapses the field.

Just tried but I can't reproduce that one yet. My version of the admin theme here is a little newer so maybe that's the difference. I'll get these files together and commit them shortly just in case that's it. 

Another point (already mentioned earlier). In the setup menu dropdowns the translation isn't available. Only 'languages' is translated.

Thanks, I found the problem and fixed it. It will appear in the next commit. 

yes, a lot of time spent to try other fonts... Wouldn't it be easier to stay with 'Arial, Helvetica, sans-serif'?

Yes it may come to that. I have no problem with Arial, but I've also become very attached to Arimo and so want to exhaust all options before giving up on it. 

Link to comment
Share on other sites

I've committed some updates to the admin theme, including a bump of the font size up a bit to hopefully look better in Chrome/Windows. I'm curious if you guys running on that platform find it makes an improvement?

  • Like 1
Link to comment
Share on other sites

hi Ryan

tested the 2.3.7 version. In my eyes it looks nice on Win 7 in FX and Chrome. With FX it looks a little bit sharper. Also the font-size is good (not too small). The dropdown translation is fixed too.

post-1027-0-82903400-1384446115_thumb.jp

Translation questions:

what about making the ProcessLanguageTranslator module translatable (the text on top and the save button)?

post-1027-0-55159000-1384446954_thumb.jp

under access/user the column titles?

post-1027-0-18702600-1384447040_thumb.jp

under access/user/username the Roles section is not translated?

post-1027-0-26699400-1384447267_thumb.jp

Link to comment
Share on other sites

Thanks I was able to reproduce as well. Looks like I need to wrap the mouseleave() in a 250ms setTimeout() for Firefox, at least that seems to fix it here. I'll commit this update tomorrow. 

Late to the party, but may I suggest superfish.js? Does wonders for my dropdowns!

I'm so excited about the changes you've made, Ryan! Can't wait to try it out.

  • Like 1
Link to comment
Share on other sites

Feasibly we could switch to a different icon set in the future, and the icon-iconname format strikes me as more portable? The fa-iconname format seems to be branded specific to Font Awesome.

Ryan: are you certain about this? I was just about to post that this little CSS rule in Font Awesome stylesheet is an awesome (pun intended) example of doing it wrong:

[class^="icon-"],[class*=" icon-"]{...

.. but based on above comments it seems that they've already realized and fixed this. The issue is that these aren't exactly "unique" class names and will collide with classes used in some existing Process modules (or, for that matter, in any other external stylesheets / styles that are also doing it wrong), which is not very nice behavior.

Sure, another approach would be to fix any collisions elsewhere and keep these rules as-is, but that feels kind of wrong, considering that this is apparently supposed to be a drop-in will-work-out-of-the-box font library (or whatever the correct term here would be.)

So.. perhaps we could implement fa-* style class names after all -- or if that seems too fa-specific, re-brand them to pw-* or something else "vendor-specific"? :)

  • Like 1
Link to comment
Share on other sites

Also wanted to note taht the new Admin Theme is broken on many things (especially the new dropdown) on IE8 and not that usable/nice. Not sure how that matters at all, but we still have clients that use IE8.

  • Like 1
Link to comment
Share on other sites

I apologize if this has already been addressed (haven't had time to read through the whole 10 page thread)... But I noticed that the page name auto-fill when creating a new page is no longer working. I am using Chrome on Mac. I checked the Chrome dev tools Console to see if there were any JS errors but didn't find anything.

Is this something on my end or is this functionality temporarily disabled in the dev branch?

Link to comment
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...