Joomla! Issue Tracker | Joomla! CMS #33776 - [4.0] Add Joomla style tooltip in com

? ?

Fixed in Code Base
13 May 2021
Medium
Build: 4.0-dev
# 33776
Diff
rjharishabh:tooltip-com-messages

Pending

User tests: Successful: Unsuccessful:

rjharishabh
11 May 2021

Pull Request for Issue # .

Summary of Changes

Add div role="tooltip"

Testing Instructions

Dashboard > Users > Messaging > Private Messages
If your site doesn't have any redirects, please create some redirects using New

Hover on icons present in Read column
Apply PR
Again hover and see the difference

Actual result BEFORE applying this Pull Request

Non-Joomla style tooltip

Expected result AFTER applying this Pull Request

Joomla style tooltip

Documentation Changes Required

560571f 11 May 2021

add joomla style tooltip

rjharishabh - open - 11 May 2021

rjharishabh - change - 11 May 2021

Status

New

⇒

Pending

joomla-cms-bot - change - 11 May 2021

Category

⇒

Administration com_messages

8e691d2 11 May 2021

php-cs

rjharishabh - change - 11 May 2021

Labels

Added: ?

3a69af6 11 May 2021

php cs more happy

rjharishabh - change - 11 May 2021

Title

…

[4.0] Add Joomla style tooltip in com-messages

[4.0] Add Joomla style tooltip in com_messages

rjharishabh - edited - 11 May 2021

b433579 11 May 2021

indentation corrected

7018148 11 May 2021

indent corrected again

Quy - test_item - 11 May 2021 - Tested successfully

Quy - comment - 11 May 2021

I have tested this item ✅ successfully on 7018148

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33776.}

sandramay0905 - test_item - 12 May 2021 - Tested successfully

sandramay0905 - comment - 12 May 2021

I have tested this item ✅ successfully on 7018148

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33776.}

richard67 - change - 12 May 2021

Status

Pending

⇒

Ready to Commit

richard67 - comment - 12 May 2021

RTC

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/33776.}

infograf768 - close - 13 May 2021

infograf768 - merge - 13 May 2021

infograf768 - change - 13 May 2021

Status	Ready to Commit	⇒	Fixed in Code Base
Closed_Date	0000-00-00 00:00:00	⇒	2021-05-13 06:45:32
Closed_By		⇒	infograf768
Labels	Added: ?

infograf768 - comment - 13 May 2021

tks

rjharishabh - comment - 13 May 2021

Thanks for merging

infograf768 - comment - 13 May 2021

@rjharishabh

If you have time, you maybe could also change title to tooltip for the Managers where we have

Using (here for articles) something like

										<?php endif; ?>
										<?php if ($canEdit || $canEditOwn) : ?>
											<a href="<?php echo Route::_('index.php?option=com_content&task=article.edit&id=' . $item->id); ?>" aria-labelledby="edit-<?php echo (int) $item->id; ?>">
												<?php echo $this->escape($item->title); ?></a>
											<div role="tooltip" id="edit-<?php echo (int) $item->id; ?>"><?php echo Text::_('JACTION_EDIT'); ?> <?php echo $this->escape($item->title); ?></div>
										<?php else : ?>

to get