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] Add missing Toggle inline help button to plugin modal / dialog #43296

Merged
merged 4 commits into from
May 21, 2024

Conversation

Ruud68
Copy link
Contributor

@Ruud68 Ruud68 commented Apr 18, 2024

Pull Request for Issue #43289 (comment) .

Summary of Changes

plugin modal was missing Toggle inline help button

Testing Instructions

a. add a description to a field in the file ./plugins/system/redirect/redirect.xml
e.g. to this field:

				<field
					name="collect_urls"
					type="radio"
					layout="joomla.form.field.radio.switcher"
					label="PLG_SYSTEM_REDIRECT_FIELD_COLLECT_URLS_LABEL"
					description="PLG_SYSTEM_REDIRECT_FIELD_COLLECT_URLS_LABEL"
					default="1"
					filter="integer"
					>
					<option value="0">JDISABLED</option>
					<option value="1">JENABLED</option>
				</field>

(line 6: description = has been added)

  1. disable the system redirect plugin
    image

  2. goto com_redirect

  3. in the alert telling you that the system redirect plugin is disabled, click on the link > the plugin will open in a modal
    image

Actual result BEFORE applying this Pull Request

The modal had three buttons: Save, Save & Close and Close
image

Expected result AFTER applying this Pull Request

The modal has 4 buttons: Save, Save & Close, Close and Toggle inline help
image

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@Ruud68 Ruud68 changed the title AddToggle inline help button to plugin modal / dialog [5.1] Add missing Toggle inline help button to plugin modal / dialog Apr 18, 2024
@dautrich
Copy link

I have tested this item 🔴 unsuccessfully on 9ed546c


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

1 similar comment
@brianteeman
Copy link
Contributor

I have tested this item 🔴 unsuccessfully on 9ed546c


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

@brianteeman
Copy link
Contributor

Are you sure that is a good test instruction. That plugin doesnt show the toggle inline help button in a normal view either

@brianteeman
Copy link
Contributor

I have tested this item ✅ successfully on 9ed546c

after testing with a plugin that does have inline help I am changing my test result to successfull.

Please provide better testing instructions!!


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

@dautrich
Copy link

@brianteeman How did you test? Which plugin?

@brianteeman
Copy link
Contributor

i edited the code so that I could test the sef plugin

@Ruud68
Copy link
Contributor Author

Ruud68 commented Apr 19, 2024

Sorry everybody, instructions updated. Toggle inline Help button is hidden when there are no descriptions in the XML file. I have added a step to add a description to the redirect plugin.
I also added visuals for what and where to click / what it will show.

Hope this makes it clear. Sorry for the confusion!

@Quy
Copy link
Contributor

Quy commented Apr 19, 2024

I have tested this item ✅ successfully on 9ed546c


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

@Quy
Copy link
Contributor

Quy commented Apr 19, 2024

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Apr 19, 2024
@Ruud68
Copy link
Contributor Author

Ruud68 commented Apr 22, 2024

Thanks for testing everybody! Appreciate you taking the time for it.

@HLeithner HLeithner added the bug label Apr 24, 2024
@LadySolveig
Copy link
Contributor

Thank you for this great idea!
As it is not yet available in other modals either, such as article new/edit or contact new/edit, I see it as a new feature. Since we only deal with bug fixes in 5.1, I will pass it on to the 5.2 team @pe7er @woluweb
Could you imagine adding it to the other components as well @Ruud68 ? That would be wonderful!

@LadySolveig LadySolveig changed the base branch from 5.1-dev to 5.2-dev May 11, 2024 11:27
@LadySolveig LadySolveig changed the title [5.1] Add missing Toggle inline help button to plugin modal / dialog [5.2] Add missing Toggle inline help button to plugin modal / dialog May 11, 2024
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators NPM Resource Changed This Pull Request can't be tested by Patchtester PR-5.2-dev Unit/System Tests labels May 11, 2024
@brianteeman
Copy link
Contributor

Could you imagine adding it to the other components as well @Ruud68 ? That would be wonderful!

What other components?

@joomla-cms-bot joomla-cms-bot removed Language Change This is for Translators Unit/System Tests NPM Resource Changed This Pull Request can't be tested by Patchtester labels May 15, 2024
@LadySolveig
Copy link
Contributor

We have also dialogs for adding and editing contacts, articles and it would be nice if we could have it here also.

grafik
grafik

Not sure if I am missing something, where this could also be useful.

@brianteeman
Copy link
Contributor

Sorry I must have confused this PR with another one as I was sure I had seen changed across all modals

@pe7er pe7er self-assigned this May 21, 2024
@pe7er pe7er enabled auto-merge (squash) May 21, 2024 08:45
@pe7er pe7er merged commit 8ecc131 into joomla:5.2-dev May 21, 2024
3 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label May 21, 2024
@pe7er
Copy link
Contributor

pe7er commented May 21, 2024

Thanks @Ruud68 ! It's a nice improvement!
Actually, we would like to have this improvement in all Modal views,
so we've created a new issue for that: #43501
Could you please help out with that?

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

Successfully merging this pull request may close these issues.

None yet

9 participants