Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.2] Filter bug with custom admin menu item #44407

Open
brianteeman opened this issue Nov 4, 2024 · 5 comments
Open

[5.2] Filter bug with custom admin menu item #44407

brianteeman opened this issue Nov 4, 2024 · 5 comments

Comments

@brianteeman
Copy link
Contributor

Steps to reproduce the issue

Create an admin menu module
Create a link of type menu items
Publish this additional admin menu module in position menu
Click on the newly created menu item link

Expected result

The site/admin filter works

Actual result

The filter never changes from site

System information (as much as possible)

Additional comments

@hytsch
Copy link

hytsch commented Nov 8, 2024

Too bad this bug was not fixed with 5.2.1. As long as the bug persists, we have to stay with 5.1 ;(

@alikon
Copy link
Contributor

alikon commented Nov 8, 2024

should this be solved with #44426 ?

@brianteeman
Copy link
Contributor Author

No. #44426 is to fix the code Hannes removed in error from com_content. Possibly this issue can be fixed with another PR to fix the code removed from com_modules

@Hackwar Hackwar added the bug label Nov 28, 2024
@brianteeman
Copy link
Contributor Author

Found the problem - it doesnt switch because the second filter to select the menu needs to be unselected

@brianteeman
Copy link
Contributor Author

Perhaps someone who understand the javascript can fix this.
Basically what is needed is for the second filter to be set to "- Select Menu -" whenever the first filter is changed from Site to Admin and vice versa

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants