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.3] Help button link urls #40268

Closed
ceford opened this issue Mar 31, 2023 · 2 comments
Closed

[4.3] Help button link urls #40268

ceford opened this issue Mar 31, 2023 · 2 comments

Comments

@ceford
Copy link
Contributor

ceford commented Mar 31, 2023

Steps to reproduce the issue

In 4.3.0-...dev configuration.php set the minor version of $helpurl in configuration.php to 2 (hard code it). Select Content / Articles and click the Help button.

Note that this is a different problem from #39818

Expected result

A Help page.

Actual result

The Joomla Docs site Home page.

System information (as much as possible)

4.3.0-beta5-dev

Additional comments

I can change the Help button url using the browser Developer Tools. This is what I found:

In 4.2 the button code looks like this:

<button class="button-help btn btn-info" type="button" onclick="Joomla.popupWindow('https:\/\/help.joomla.org\/proxy?keyref=Help42:Users&amp;lang=en', 'Help', 700, 500, 1)" title="Opens in a new window">
    <span class="icon-question" aria-hidden="true"></span>
    Help</button>

In 4.3 it looks like this:

<button class="button-help btn btn-info js-toolbar-help-btn" data-url="https:\/\/help.joomla.org\/proxy?keyref=Help42:Users&amp;lang=en" data-title="Help" data-width="700" data-height="500" data-scroll="1" type="button" title="Opens in a new window">
    <span class="icon-question" aria-hidden="true"></span>
    Help</button>

For the latter, the back slashes in the url need to be removed. With them removed with the browser Developer Tools the Help link works correctly.

@dgrammatiko
Copy link
Contributor

Check #40270

@alikon
Copy link
Contributor

alikon commented Mar 31, 2023

closing as we have a pr #40270

@alikon alikon closed this as completed Mar 31, 2023
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

4 participants