Skip to content
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

Isis Retina - Dropdown icons fix #12527

Closed
wants to merge 290 commits into from
Closed

Isis Retina - Dropdown icons fix #12527

wants to merge 290 commits into from

Conversation

ciar4n
Copy link
Contributor

@ciar4n ciar4n commented Oct 23, 2016

Pull Request for Issue #12500 .

Summary of Changes

Removes sprite image from dropdown on retina displays.

Testing Instructions

Navigate to article edit. If using a Retina display you should see triple icons on dropdown selects which is incorrect. If not on a retina display, zoom in a few notches on your browser to replicate the issue.

Before Patch
retina1

After Patch
retina2

Documentation Changes Required

None

Michael Babker and others added 30 commits June 11, 2016 21:53
csthomas and others added 23 commits October 18, 2016 19:32
* Extend test for document html links

* Fix mergeHeadData method for html document->_links
…mit (solve #12122) (#12137)

* ismoo parameter need to exist after form submit

* int

* default to 1
* Replaced rand() with mt_rand() projectwide

* session package is from the framework so we need to revert that change at this point

* fof is an external lib so we need to revert that change at this point

* fof is an external lib so we need to revert that change at this point
* JLayoutFile wrong type hint

* CS
* cs review for the editor plugin

* cs

* Update image.php

* Call to undefined method stdClass::get()
* Installation flat repaint - Initial commit

* Install flat repaint - Initial commit
* Update polyfill.event-uncompressed.js

* Update polyfill.event.js

* update classlist polyfill

* Update polyfill.classlist.js

* minor changes

* minor changes in doc block
Just a small typo in the comment.
@ciar4n ciar4n closed this Oct 23, 2016
@ciar4n ciar4n deleted the isis-retina branch October 23, 2016 15:19
@ciar4n ciar4n restored the isis-retina branch October 23, 2016 15:19
@ciar4n
Copy link
Contributor Author

ciar4n commented Oct 23, 2016

Closed due to incorrect base

@zero-24
Copy link
Contributor

zero-24 commented Oct 23, 2016

Just a tip for the future if you just send it against the wrong branch you can change the base branch with the edit button on the top right position. ;)

@ciar4n
Copy link
Contributor Author

ciar4n commented Oct 31, 2016

Thank you @zero-24 Never noticed that before! Learn something new every day :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.