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

[4.0] media manager transparent preview #30870

Merged
merged 1 commit into from
Oct 5, 2020
Merged

Conversation

brianteeman
Copy link
Contributor

@brianteeman brianteeman commented Oct 2, 2020

The media manager uses a background of checked squares in the thumbnail view which is useful when you have a transparent image such as the joomla_black.png

However in the preview it uses a solid white background which hides most of this image.

This pr replaces the white background with the checked squares

Dont forget to run npm ci

Before

image

After

image

The media manager uses a background of checked squares in the thumbnail view which is useful when you have a transparent image such as the joomla_black.png

However in the preview it uses a solid white background which hides most of this image.

This pr replaces the white background with the checked squares

Dont forget to run npm ci
@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.0-dev labels Oct 2, 2020
@Quy
Copy link
Contributor

Quy commented Oct 2, 2020

I have tested this item ✅ successfully on b12d7ec


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

1 similar comment
@jwaisner
Copy link
Member

jwaisner commented Oct 4, 2020

I have tested this item ✅ successfully on b12d7ec


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

@joomla-cms-bot joomla-cms-bot removed the NPM Resource Changed This Pull Request can't be tested by Patchtester label Oct 4, 2020
@jwaisner
Copy link
Member

jwaisner commented Oct 4, 2020

RTC


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

@joomla-cms-bot joomla-cms-bot added RTC This Pull Request is Ready To Commit and removed PR-4.0-dev labels Oct 4, 2020
@Quy Quy added the PR-4.0-dev label Oct 4, 2020
@HLeithner HLeithner merged commit c4db467 into joomla:4.0-dev Oct 5, 2020
@HLeithner
Copy link
Member

Thanks

@joomla-cms-bot joomla-cms-bot added NPM Resource Changed This Pull Request can't be tested by Patchtester and removed RTC This Pull Request is Ready To Commit labels Oct 5, 2020
@HLeithner HLeithner added this to the Joomla 4.0 milestone Oct 5, 2020
@brianteeman
Copy link
Contributor Author

thanks

@brianteeman brianteeman deleted the yk16 branch October 5, 2020 11:07
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
The media manager uses a background of checked squares in the thumbnail view which is useful when you have a transparent image such as the joomla_black.png

However in the preview it uses a solid white background which hides most of this image.

This pr replaces the white background with the checked squares
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.

None yet

5 participants