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

[4.4] Fix help icon not opening help screen #42820

Open
wants to merge 3 commits into
base: 4.4-dev
Choose a base branch
from

Conversation

Quy
Copy link
Contributor

@Quy Quy commented Feb 15, 2024

Pull Request for Issue #42818 .

Summary of Changes

Help icon fails to open help screen when clicked on the question mark.

Thanks @dgrammatiko for the solution.

Testing Instructions

Go to any screen which has a help screen, for example Content -> Articles.
Click on the text "Help" of the help toolbar button and you will see the appropriate help screen. However, if you click on the question mark to the left of the text you will get a much larger blank window with "about:blank" in the address bar.

Download the prebuilt package.
or
Apply PR and run npm run build:js

@ChristineWk
Copy link

I have tested this item ✅ successfully on 7a13d66

with NPM Prebuilt Package


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

@viocassel
Copy link
Contributor

I have tested this item ✅ successfully on 7a13d66


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

@Quy
Copy link
Contributor Author

Quy commented Feb 15, 2024

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Feb 15, 2024
@C-Lodder
Copy link
Member

C-Lodder commented Feb 16, 2024

Please don't use CSS to fix a JS problem

You should be using currentTarget, rather than target instead:

https://github.com/joomla/joomla-cms/blob/4.4-dev/build/media_source/legacy/js/toolbar.es5.js#L46

E.g you want to target the element that the event is attached to (button), rather than trying to target the element that you clicked, which in this case (the icon), has no event attached to it.

@Quy Quy removed the RTC This Pull Request is Ready To Commit label Feb 16, 2024
@C-Lodder
Copy link
Member

I have tested this item ✅ successfully on 92a1cf4


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

1 similar comment
@viocassel
Copy link
Contributor

I have tested this item ✅ successfully on 92a1cf4


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

@Quy
Copy link
Contributor Author

Quy commented Feb 16, 2024

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Feb 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug NPM Resource Changed This Pull Request can't be tested by Patchtester PR-4.4-dev RTC This Pull Request is Ready To Commit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants