? Pending

User tests: Successful: Unsuccessful:

avatar ciar4n
ciar4n
11 Apr 2020

Pull Request for Issue #28648, #28461, #23451.

Summary of Changes

Rewrites choices CSS

  • Reduced CSS
  • Simplifies styling
  • Removes 'Press to Select' text
  • Matches style with non choices select fields
  • Removes placholder text if 1+ item selected.

Testing Instructions

Apply this patch and run node build.js --compile-css for updating the changed SCSS. Check choices select fields (searchtools / new article -> category)

Before

image
image

After

image
image

avatar ciar4n ciar4n - open - 11 Apr 2020
avatar ciar4n ciar4n - change - 11 Apr 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 Apr 2020
Category Administration Templates (admin)
avatar alikon
alikon - comment - 11 Apr 2020

not sure if my test cover what you have fixed here anyway
i'm in com_modules
selecting some of those and then click on batch and then look at Set Position
Screenshot from 2020-04-11 22-08-42

avatar ciar4n
ciar4n - comment - 11 Apr 2020

@alikon I believe that issue always existed. The same modal before this PR...

image

Possibly related to #28196

avatar ciar4n ciar4n - change - 11 Apr 2020
The description was changed
avatar ciar4n ciar4n - edited - 11 Apr 2020
avatar richard67
richard67 - comment - 11 Apr 2020

I have tested this item successfully on e8a0de7


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

avatar richard67 richard67 - test_item - 11 Apr 2020 - Tested successfully
avatar richard67
richard67 - comment - 11 Apr 2020

I've tested also RTL ... I hope I haven't missed anything. Maybe @infograf768 could test, too, so we can be sure it is ok for RTL?

avatar brianteeman
brianteeman - comment - 11 Apr 2020

The things I tried to fix in #28461 are not fixed/included

  1. The radius of the corners is different
  2. the color of the text is different

image

  1. The focus ring is different

image

Is this intended
image

avatar richard67
richard67 - comment - 11 Apr 2020

Seems I need glasses.

avatar richard67
richard67 - comment - 11 Apr 2020

I have not tested this item.


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

avatar richard67 richard67 - test_item - 11 Apr 2020 - Not tested
avatar ciar4n ciar4n - change - 11 Apr 2020
Labels Added: ?
avatar ciar4n
ciar4n - comment - 11 Apr 2020

@brianteeman Should be fine now (267c7fa).

avatar brianteeman
brianteeman - comment - 11 Apr 2020

border radius and focus all good now.

The text color is still different
for choices it is the placeholder that needs to be changed
image

avatar ciar4n
ciar4n - comment - 11 Apr 2020

What browser are you using?.. color is the same but browser style applying opacity. Thought i reset it but obviously not.

avatar brianteeman
brianteeman - comment - 11 Apr 2020

chrome on windows

avatar ciar4n
ciar4n - comment - 11 Apr 2020

Thx. I'll check again in the morn.

avatar infograf768
infograf768 - comment - 12 Apr 2020

I guess should also be tested with Cassiopea

avatar ciar4n
ciar4n - comment - 12 Apr 2020

Placeholder color should be now correct.

All current changes are only made in Atum

avatar brianteeman
brianteeman - comment - 12 Apr 2020

awesome sauce - i can confirm the placeholder is correct now

avatar ciar4n ciar4n - change - 12 Apr 2020
The description was changed
avatar ciar4n ciar4n - edited - 12 Apr 2020
avatar wilsonge wilsonge - change - 12 Apr 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-04-12 17:22:36
Closed_By wilsonge
avatar wilsonge wilsonge - close - 12 Apr 2020
avatar wilsonge wilsonge - merge - 12 Apr 2020
avatar wilsonge
wilsonge - comment - 12 Apr 2020

Thankyou very much @ciar4n !!

avatar infograf768
infograf768 - comment - 13 Apr 2020

@ciar4n
See glip chat for RTL correction.

avatar ciar4n
ciar4n - comment - 13 Apr 2020

@infograf768 PR created... #28676

Add a Comment

Login with GitHub to post a comment