? ? Pending

User tests: Successful: Unsuccessful:

avatar rjharishabh
rjharishabh
13 May 2021

Summary of Changes

Change title to tooltip,
Add aria-labelledby in a

Testing Instructions

Go to the articles page

Please test with both RTL and LTR

Actual result BEFORE applying this Pull Request

articles-before

Expected result AFTER applying this Pull Request

articles-after

Documentation Changes Required

No

avatar rjharishabh rjharishabh - open - 13 May 2021
avatar rjharishabh rjharishabh - change - 13 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 13 May 2021
Category Administration com_content
avatar rjharishabh rjharishabh - change - 14 May 2021
Title
4.0] Add Joomla style tooltip in articles
[4.0] Add Joomla style tooltip in articles
avatar rjharishabh rjharishabh - edited - 14 May 2021
avatar sandramay0905 sandramay0905 - test_item - 14 May 2021 - Tested unsuccessfully
avatar sandramay0905
sandramay0905 - comment - 14 May 2021

I have tested this item ? unsuccessfully on af07820

Position of tooltip is different. With pr below end of link, without pr below mouse position:

With PR Without PR
Screen Shot 2021-05-14 at 08 51 20 Screen Shot 2021-05-14 at 08 52 27

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33839.
avatar rjharishabh
rjharishabh - comment - 14 May 2021

@sandramay0905 I think it's by design

avatar ceford ceford - test_item - 14 May 2021 - Tested successfully
avatar ceford
ceford - comment - 14 May 2021

I have tested this item successfully on af07820

The tooltip is distinct and reasonably well-placed in LTR and RTL. I think exact styling is a separate issue. For example, the tooltip shows/disapperas/shows on traversing a title broken over two lines.


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

avatar sandramay0905 sandramay0905 - test_item - 14 May 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 14 May 2021

I have tested this item successfully on af07820

Changed the result by comments above.


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

avatar rjharishabh rjharishabh - change - 14 May 2021
The description was changed
avatar rjharishabh rjharishabh - edited - 14 May 2021
avatar richard67 richard67 - change - 14 May 2021
Status Pending Ready to Commit
Labels Added: ?
avatar richard67
richard67 - comment - 14 May 2021

RTC


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

avatar ceford
ceford - comment - 14 May 2021

As this pr mentions Joomla style tooltip may I ask where this is documented? There are several ways to implement tooltips and I missed this one.


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

avatar rjharishabh
rjharishabh - comment - 14 May 2021

As this pr mentions Joomla style tooltip may I ask where this is documented? There are several ways to implement tooltips and I missed this one.

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

Joomla style tooltip means tooltip style used in Joomla 4 now

I will change the title

avatar rjharishabh rjharishabh - change - 14 May 2021
Title
[4.0] Add Joomla style tooltip in articles
[4.0] Change tooltip style in articles
avatar rjharishabh rjharishabh - edited - 14 May 2021
avatar ceford
ceford - comment - 14 May 2021

You misunderstood! I am actually looking for the documentation that describes how to use Tooltips in Joomla. This has changed since Joomla 3 and I need to update some ancillary documentation. Your pr title was finr.

avatar rdeutz rdeutz - change - 14 May 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-05-14 09:51:59
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 14 May 2021
avatar rdeutz rdeutz - merge - 14 May 2021
avatar infograf768
infograf768 - comment - 14 May 2021

Thanks
Please do same patch for other managers.

avatar brianteeman
brianteeman - comment - 14 May 2021

It is absolutely not necessary to change titles to tooltips. It is a pointless exercise that would need to be done in a gazillion places.

The only time we should be using the black tooltips is as a replacement for the bootstrap tooltips. They are NOT a replacement for titles.

This PR should be reverted as its not correct.

avatar ceford
ceford - comment - 14 May 2021

I stopped using titles and tooltips some years ago after reading somewhere that they had accessibility issues (icons too). It would be nice to have some guidance (documentation) on where each could/should be/not be used in Joomla 4. I could write something if nothing exists but don't what the acceptable practice should be.

avatar brianteeman
brianteeman - comment - 14 May 2021

There is nothing wrong with titles. Used correctly they are perfectly accessible

avatar infograf768
infograf768 - comment - 14 May 2021

The point is not so much accesibility here but to get a readable tip

avatar brianteeman
brianteeman - comment - 14 May 2021

why? it serves zero purpose other than to clutter the ui

avatar brianteeman
brianteeman - comment - 14 May 2021

The point is not so much accesibility here but to get a readable tip

If you are happy that this breaks accessibility then what has been the point in the last three years work

Add a Comment

Login with GitHub to post a comment