Skip to content

Conversation

@chmst
Copy link
Contributor

@chmst chmst commented May 16, 2022

Pull Request for Issue # .

Summary of Changes

The permissions overview contains 5 columns which contain always the same value because these are global permissions.
This PR extracts global permissions from the list of possible actions and displays the result in a single line.

It does not resolve the issue #23628. Horizontal scrolling appears in all screens where the table heading enforces horizontal scrolling, not only here. As soon as the table width cannot be reduced to fit the viewport, a horizontal scrollbar appears.
We resolve this in other tables by hiding some olumns, but on the permissions this is not possible.

Testing Instructions

Have a look on the permissions of a user before and after the patch.

Actual result BEFORE applying this Pull Request

grafik

Expected result AFTER applying this Pull Request

grafik

Documentation Changes Required

yes

@ChristineWk
Copy link

I have tested this item ✅ successfully on b71a3a6


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

1 similar comment
@viocassel
Copy link
Contributor

I have tested this item ✅ successfully on b71a3a6


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

@chmst chmst changed the title Reduce number of permission columns [4.2] Reduce number of permission columns May 16, 2022
@richard67
Copy link
Member

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 17, 2022
@richard67
Copy link
Member

I've restored the previous results from human tests in the issue tracker since the commit after that was just a code style change regarding white space.

@roland-d roland-d merged commit 4374661 into joomla:4.2-dev May 17, 2022
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label May 17, 2022
@roland-d
Copy link
Contributor

Thanks everybody

@brianteeman
Copy link
Contributor

Unfortunately this PR is not correct and creates errors and misleading information

See #38781 (comment)
and #38830

@chmst chmst deleted the users-debuguser-columns branch July 13, 2025 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants