? ? Pending

User tests: Successful: Unsuccessful:

avatar bembelimen
bembelimen
2 Jan 2021

Pull Request for Issue #29910 .

Summary of Changes

Fix the width of the position select in modules batch

Testing Instructions

Check the batch in com_modules

Actual result BEFORE applying this Pull Request

Select for positon is small

Expected result AFTER applying this Pull Request

Select for positon is 100%

avatar bembelimen bembelimen - open - 2 Jan 2021
avatar bembelimen bembelimen - change - 2 Jan 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Jan 2021
Category Administration com_modules
avatar richard67
richard67 - comment - 2 Jan 2021

I have tested this item successfully on 4e92ad6


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

avatar richard67 richard67 - test_item - 2 Jan 2021 - Tested successfully
avatar infograf768
infograf768 - comment - 2 Jan 2021

I have tested this item successfully on 4e92ad6


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

avatar infograf768 infograf768 - test_item - 2 Jan 2021 - Tested successfully
avatar richard67 richard67 - change - 2 Jan 2021
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 2 Jan 2021

RTC


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

avatar richard67 richard67 - change - 2 Jan 2021
Labels Added: ? ?
avatar infograf768
infograf768 - comment - 2 Jan 2021

This indeed does the job in the description and that's why I tested it OK.

But I think we have a UX issue here.
We should have a - Keep Original Position - display when we use the small x to kill the :: None :: option.
In fact it should be the default and not :: None ::

I mean when we empty the input field.
Screen Shot 2021-01-02 at 11 09 37

avatar wilsonge wilsonge - change - 2 Jan 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-01-02 20:38:36
Closed_By wilsonge
Labels
avatar wilsonge wilsonge - close - 2 Jan 2021
avatar wilsonge wilsonge - merge - 2 Jan 2021
avatar wilsonge
wilsonge - comment - 2 Jan 2021

@infograf768 id agree with that. You able to do the patch?

Add a Comment

Login with GitHub to post a comment