joomla-cms icon indicating copy to clipboard operation
joomla-cms copied to clipboard

[5.2] [Guided tours] Add a What's New tour when updating to Joomla 5.2

Open obuisard opened this issue 1 year ago • 14 comments

Summary of Changes

This PR assumes PR #43814 has been merged. It add a tour that highlights new features introduced in Joomla 5.2.

Testing Instructions

The tour will not autostart until PR #43814 has been merged. It is possible to find the tour in the list of tours at the top of the Joomla dashboard and run it manually. Make sure you see images when running the tour (on the first step only) and when editing the tour (System -> Manage -> Guided tours).

Actual result BEFORE applying this Pull Request

No tour highlighting new features for Joomla 5.2.

Expected result AFTER applying this Pull Request

What’s New in Joomla 5.2! tour is available.

image

image

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

obuisard avatar Aug 23 '24 03:08 obuisard

Using images in this way means that they are not translatable. Is that a good idea?

brianteeman avatar Aug 23 '24 08:08 brianteeman

Is the intention for this to replace Post-Installation messages?

brianteeman avatar Aug 23 '24 08:08 brianteeman

Regarding "untranslatable" pictures - good point. I would prefer that all possible pictures for translation were like that.

Something like the title enhancedtourheader_en-GB.png. If the system does not find a picture with the title enhancedtourheader_YOUR_LANGUAGE_CODE.png, the standard en-GB is loaded.

Kostelano avatar Aug 23 '24 08:08 Kostelano

Neither the sql for tours or steps adds a value for ordering.

Why does this matter? The module displays tours ordered by ordering. So the whats new is always at the top of the list where it probably should be at the bottom next to the welcome to joomla tour so that it doesnt always display in the dropdown

image

brianteeman avatar Aug 23 '24 08:08 brianteeman

Regarding "untranslatable" pictures - good point. I would prefer that all possible pictures for translation were like that.

Something like the title enhancedtourheader_en-GB.png. If the system does not find a picture with the title enhancedtourheader_YOUR_LANGUAGE_CODE.png, the standard en-GB is loaded.

not that simple as we dont have a method for you to add an image file into the language pack

brianteeman avatar Aug 23 '24 08:08 brianteeman

Is the intention for this to replace Post-Installation messages?

The idea is for the What's new tours to highlight new features of a release (major or minor) on an update, of interest to the user, which is not quite the same as the post-installation messages, which targets admin, pointing that there are changes in htaccess... At least, that is how I see it.

obuisard avatar Aug 24 '24 14:08 obuisard

Using images in this way means that they are not translatable. Is that a good idea?

No, but we could not find anything to hint us on how to do this (other than how the documentation screens are handled).

obuisard avatar Aug 24 '24 14:08 obuisard

Neither the sql for tours or steps adds a value for ordering.

Why does this matter? The module displays tours ordered by ordering. So the whats new is always at the top of the list where it probably should be at the bottom next to the welcome to joomla tour so that it doesnt always display in the dropdown

All tours and steps get an ordering value of 0 on installation. Then values change once the user starts ordering tours. Do you mean we should check all ordering values of the tables, get the highest number, then use that number + 1 in our SQL scripts?

obuisard avatar Aug 24 '24 14:08 obuisard

Regarding "untranslatable" pictures - good point. I would prefer that all possible pictures for translation were like that. Something like the title enhancedtourheader_en-GB.png. If the system does not find a picture with the title enhancedtourheader_YOUR_LANGUAGE_CODE.png, the standard en-GB is loaded.

not that simple as we dont have a method for you to add an image file into the language pack

I think adding the language suffix is a good idea, but then we need to add the logic into this PR to find images according to languages. Images ultimately should still remain in the /media folder

obuisard avatar Aug 24 '24 14:08 obuisard

All tours and steps get an ordering value of 0 on installation. Then values change once the user starts ordering tours.

No that is not correct. You can see that the ordering is set on installation here

https://github.com/joomla/joomla-cms/blob/76a2903de3bd0b10a99619d31a7ad0cd8b395065/installation/sql/mysql/extensions.sql#L987-L999

brianteeman avatar Aug 24 '24 15:08 brianteeman

Using images in this way means that they are not translatable. Is that a good idea?

No, but we could not find anything to hint us on how to do this (other than how the documentation screens are handled).

I dont have any suggestion to make this work but having english screenshots in an arabic installation is not useful

brianteeman avatar Aug 24 '24 15:08 brianteeman

All tours and steps get an ordering value of 0 on installation. Then values change once the user starts ordering tours.

No that is not correct. You can see that the ordering is set on installation here

OMG I should have checked, I forgot about it... I should know, I fixed those SQL lines back in Joomla 4.3 :-) Anyway, this tour is not installed on a new installation, which means, all ordering will/may be different once we add this tour (and many more can have been added in between). So I don't see what ordering should be set here.

obuisard avatar Aug 24 '24 16:08 obuisard

Using images in this way means that they are not translatable. Is that a good idea?

No, but we could not find anything to hint us on how to do this (other than how the documentation screens are handled).

I dont have any suggestion to make this work but having english screenshots in an arabic installation is not useful

I agree completely. I put this PR in draft so we can stir some ideas about that problematic...

obuisard avatar Aug 24 '24 16:08 obuisard

It could be argued that the two screenshots do not add anything to the guided tour and can simply be removed.

brianteeman avatar Aug 24 '24 16:08 brianteeman

I have tested this item :white_check_mark: successfully on 97b4ef3b3e1a784b8f18055466df54a1ce4b6170

I have tested this successfully.. everything worked great.

NOTE: on the 2nd image, the envelope icon is hard to see for someone that sees perfectly well so definitely not good enough for those with visual impairments.


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

exlemor avatar Sep 03 '24 20:09 exlemor

NOTE: on the 2nd image, the envelope icon is hard to see for someone that sees perfectly well so definitely not good enough for those with visual impairments.

Thank you for testing Emmanuel @exlemor! I have updated the image in question.

obuisard avatar Sep 04 '24 02:09 obuisard

Using prebuilt package no "Whats new in Joomla 5.2" available:

Untitled1

"Show all tours":

Untitled2 index.php?option=com_guidedtours&view=tours shows 12 published and no unpublished tours.

ghost avatar Sep 04 '24 05:09 ghost

Using prebuilt package no "Whats new in Joomla 5.2" available:

Hello @fgsw, you have to install the update package, you cannot use the full install package. When on new installs, you get the Welcome tour. On updated site, you get the What's new tour.

I have updated the test section to add that information (sorry, it was missing).

Thanks for trying the PR out!

obuisard avatar Sep 04 '24 14:09 obuisard

I have tested this item :white_check_mark: successfully on 97b4ef3b3e1a784b8f18055466df54a1ce4b6170


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

ghost avatar Sep 06 '24 06:09 ghost

I have tested this item :white_check_mark: successfully on 97b4ef3b3e1a784b8f18055466df54a1ce4b6170

I have re-tested this successfully with the latest PR Update.zip from ~17:26 UTC of today.


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

exlemor avatar Sep 06 '24 19:09 exlemor

RTC


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

Hackwar avatar Sep 11 '24 14:09 Hackwar

Thanks @obuisard !

pe7er avatar Sep 12 '24 03:09 pe7er