Skip to content

[4.0] Menu item edit: Correcting Unassigned and Unpublished Modules assignment#27492

Merged
HLeithner merged 4 commits intojoomla:4.0-devfrom
infograf768:4.0_module_assigment
Jan 14, 2020
Merged

[4.0] Menu item edit: Correcting Unassigned and Unpublished Modules assignment#27492
HLeithner merged 4 commits intojoomla:4.0-devfrom
infograf768:4.0_module_assigment

Conversation

@infograf768
Copy link
Copy Markdown
Member

Pull Request for Issue #27491

Summary of Changes

remove hidden class when Unassigned Modules or Unpublished Modules are set to Show

Testing Instructions

See #27491

After patch

The rows are no more empty and show the Display and Published badges correctly.

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 12, 2020

It works the first time, but if you hide it again, you get a blank row.

27492

@jwaisner
Copy link
Copy Markdown
Member

I get the same result. I even tested with the "unassigned module" toggle and it did the same thing. Created empty rows if I toggled either more than once.


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

@infograf768
Copy link
Copy Markdown
Member Author

@Quy @jwaisner
Should be OK now. Please test again.

@jwaisner
Copy link
Copy Markdown
Member

I have tested this item ✅ successfully on 114971d

Both toggle functions work as expected.


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

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 13, 2020

I have tested this item ✅ successfully on 55ead68


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

@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 13, 2020

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Jan 13, 2020
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Jan 13, 2020

There is vertical spacing issue when hiding unassigned modules, but can be done in a different PR.

27492a

@HLeithner
Copy link
Copy Markdown
Member

There is vertical spacing issue when hiding unassigned modules, but can be done in a different PR.

27492a

Is this a regression? If you then you should remove the RTC tag

@infograf768
Copy link
Copy Markdown
Member Author

Will look at this tomorrow, not sure it is regression as the only changes concern class remove

@brianteeman
Copy link
Copy Markdown
Contributor

I can not replicate @Quy findings - all looks good to me

@infograf768
Copy link
Copy Markdown
Member Author

I confirm no issue here.
@HLeithner
Please merge

@HLeithner HLeithner merged commit a1d4d0d into joomla:4.0-dev Jan 14, 2020
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Jan 14, 2020
@HLeithner
Copy link
Copy Markdown
Member

Thanks

@HLeithner HLeithner added this to the Joomla 4.0 milestone Jan 14, 2020
@infograf768 infograf768 deleted the 4.0_module_assigment branch January 14, 2020 08:58
brianteeman pushed a commit to brianteeman/joomla-cms that referenced this pull request Feb 4, 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.

6 participants