Skip to content

[4.0] Fix Atum Safari not displaying svg background when using choices#29675

Merged
Quy merged 1 commit intojoomla:4.0-devfrom
infograf768:4.0_safari_background_image
Jun 17, 2020
Merged

[4.0] Fix Atum Safari not displaying svg background when using choices#29675
Quy merged 1 commit intojoomla:4.0-devfrom
infograf768:4.0_safari_background_image

Conversation

@infograf768
Copy link
Copy Markdown
Member

@infograf768 infograf768 commented Jun 17, 2020

Pull Request for Issue #29658

Summary of Changes

Safari and possibly some other browsers do not like combining background color and url of image. In our case the select-bg.svg as well as select-bg-rtl.svg
This PR separates both.

Testing Instructions

See #29658
Make sure that all caches are deleted after patching.
This needs npm to test

After patch

Screen Shot 2020-06-17 at 12 00 27

@PhilETaylor

This comment was marked as abuse.

@bonzani
Copy link
Copy Markdown

bonzani commented Jun 17, 2020

I have tested this item ✅ successfully on 00a94b6


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29675.

@infograf768 infograf768 removed NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Jun 17, 2020
@infograf768
Copy link
Copy Markdown
Member Author

rtc


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/29675.

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jun 17, 2020
@Quy Quy added this to the Joomla 4.0 milestone Jun 17, 2020
@Quy Quy merged commit 4f8a9d9 into joomla:4.0-dev Jun 17, 2020
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev and removed RTC This Pull Request is Ready To Commit labels Jun 17, 2020
@infograf768 infograf768 deleted the 4.0_safari_background_image branch June 17, 2020 13:50
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NPM Resource Changed This Pull Request can't be tested by Patchtester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants