? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
25 Dec 2020

Adds the same scss that is used in the backend for the image select field so that it is styled correctly.

To test dont forget to run npm

Before

image

After

image

avatar brianteeman brianteeman - open - 25 Dec 2020
avatar brianteeman brianteeman - change - 25 Dec 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Dec 2020
Category Front End Templates (site) NPM Change
avatar dgrammatiko
dgrammatiko - comment - 26 Dec 2020

I have tested this item successfully on 3de2174


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

Screenshot 2020-12-26 at 20 16 41

avatar dgrammatiko dgrammatiko - test_item - 26 Dec 2020 - Tested successfully
avatar gostn
gostn - comment - 27 Dec 2020

I have tested this item successfully on 3de2174


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

avatar gostn gostn - test_item - 27 Dec 2020 - Tested successfully
avatar infograf768
infograf768 - comment - 27 Dec 2020

I have tested this item ? unsuccessfully on 3de2174


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

avatar infograf768 infograf768 - test_item - 27 Dec 2020 - Tested unsuccessfully
avatar infograf768
infograf768 - comment - 27 Dec 2020

Please take off max-width: calc(50vw - 5rem);

imager_select

Same issue when editing an article when Frontend Images and Links is set.

avatar brianteeman
brianteeman - comment - 27 Dec 2020

Sorry but I see no reason to remove that. I stick by this PR as it is

avatar Quy Quy - change - 18 Feb 2021
Status Pending Ready to Commit
avatar Quy
Quy - comment - 18 Feb 2021

RTC


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

avatar laoneo
laoneo - comment - 18 Feb 2021

@brianteeman why did you add the max-width? For me it looks nicer without.

avatar brianteeman
brianteeman - comment - 18 Feb 2021

@laoneo

Adds the same scss that is used in the backend for the image select field so that it is styled correctly.

avatar laoneo laoneo - close - 18 Feb 2021
avatar laoneo laoneo - merge - 18 Feb 2021
avatar laoneo laoneo - change - 18 Feb 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-02-18 08:26:49
Closed_By laoneo
Labels Added: ? ?
avatar laoneo
laoneo - comment - 18 Feb 2021

Thanks @brianteeman for the explanation. Merged this one as it makes sense to have the same behavior on front and back end. @infograf768 can you please make a followup pr with your suggestion on the back and front end then?

avatar infograf768
infograf768 - comment - 18 Feb 2021

@laoneo
I am sure the original poster will followup. It would have been so simple to correct both in admin and site in this pr and modify the description.

Add a Comment

Login with GitHub to post a comment