-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
RETINA displays only - Dropdowns icons looks messy #12500
Comments
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
Rechecked current 3.7 branch with chrome, safari and firefox on osx and still can not replicate this |
This comment was marked as abuse.
This comment was marked as abuse.
It is a privat video ;) can you double check that the caches are clean? |
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
I've got the same issue with my MacBook Pro. So either there's a bug with the upstream Chosen code or our implementation of it. |
This comment was marked as abuse.
This comment was marked as abuse.
who told you to buy mac "retina", clearly your |
now kidding aside, so this only manisfests in new macs? |
Mine's not "new" by any means.
|
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
This comment was marked as abuse.
Shoulda held out a few weeks. Isn't this the time of year all the new stuff hits the market? |
This comment was marked as abuse.
This comment was marked as abuse.
@PhilETaylor if you were waiting one week more: http://www.macrumors.com/2016/10/22/kuo-october-2016-mac-predictions/ |
I have updated the title This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12500. |
@ciar4n can you take a look at this issue? |
Unable test at the moment but I'm guessing the issue is here which would only effect high pixel density displays.. https://github.com/joomla/joomla-cms/blob/3.7.x/media/jui/css/chosen.css#L435 If so, zooming in a couple of notches in any browser should replicate the issue? I can put in a PR to fix tomorrow. |
@ciar4n you nailed it, zoomed the page a lot in chrome and widonwss and was able to reproduce it |
@ciar4n I thought we were using the mootools icon by now for chosen... |
Thank you @andrepereiradasilva .. removing that CSS should resolve it so. Alternatively I can override it in the template.css so as to keep the chosen.css to the original. @DGT41 Yes, we are using the mootools/icomoon icon. That CSS is loading a 2x sprite for retina displays using '!important' which is overriding the CSS used to remove the sprite, hence the double icons (one the icomoon icon, the other the original sprite image). |
Closing as there is a PR #12528 for this please test on retina. |
This video is private I am not saying I dont believe you are seeing this I am just saying I On 22 October 2016 at 20:25, Phil Taylor [email protected] wrote:
Brian Teeman |
Bingo - I can replicate it now on chrome if I "massively" zoom in On 22 October 2016 at 22:40, Ciaran Walsh [email protected] wrote:
Brian Teeman |
On my phone so can't test but
http://stackoverflow.com/questions/15551287/how-to-test-a-website-for-retina-on-windows-without-an-actual-retina-display
|
Ok so we can confirm this as an issue on retina displays then.
Can this be emulated on non retina displays
|
Iirc the chosen update that I did was merged into 3.6.3 but aspects of it
were overwritten in the "flat paint" pr. So I would look at those overrides
of chosen first before looking at chosen itself
|
I'm assuming from the comments that it will be any retina device.
|
I do not have a retina screen
|
What am I supposed to be seeing as messy? On 21 October 2016 at 16:49, Valentín [email protected] wrote:
Brian Teeman |
If I wasn't clear, this happens with 3.7.x branch. |
This comment was marked as abuse.
This comment was marked as abuse.
Thanks @PhilETaylor! Awesome job, everyone! |
Steps to reproduce the issue
Create or edit an article, and see the icons on the right side for the dropdowns such as "Access" or "Status"
Expected result
Actual result
System information (as much as possible)
Using Joomla 3.7.x branch
Firefox browser
Mac OS
Additional comments
The text was updated successfully, but these errors were encountered: