? Success

User tests: Successful: Unsuccessful:

avatar JoomliC
JoomliC
15 Apr 2016

EDIT: REDO PR #9379
This PR has fixed tooltip not visible in batch modal.
But tooltip are not displayed (since 3.5.0 or before...) if module multilanguage status is enabled

Testing Instructions

  • Open batch modals (articles, menus...)
  • hover the labels and be sure the tooltip is always visible (as describe in this issue #9363 screenshots)
  • Test with and without admin module multilanguage status enabled
avatar JoomliC JoomliC - open - 15 Apr 2016
avatar JoomliC JoomliC - change - 15 Apr 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Apr 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 16 Apr 2016
Category Layout
avatar andrepereiradasilva
andrepereiradasilva - comment - 16 Apr 2016

hum ... for me the tooltip doesn't exist with or without the PR (using latest staging on chrome).

avatar JoomliC
JoomliC - comment - 16 Apr 2016

@andrepereiradasilva just tested with latest staging and fresh install on Chrome (mac), and i have the label tooltip:
capture d ecran 2016-04-16 a 14 48 24
:-|

Note: with or without patch, tooltip works for me...

avatar andrepereiradasilva
andrepereiradasilva - comment - 16 Apr 2016

ok, so clearly is something on my part.

avatar andrepereiradasilva
andrepereiradasilva - comment - 16 Apr 2016

i also tested with firefox and same problem. no tooltips in batch modals.

Either way i have some modals where the tooltip exists and it's croped (the article select modal in single article menu item), so i tested your PR in that one.

And ... it's still croped.

image

avatar JoomliC
JoomliC - comment - 17 Apr 2016

@andrepereiradasilva there's a misunderstanding of this PR... ;-)
And, this is not related to any tooltip outside of the batch modals!
In Batch modal, it's not using jform fields from components, but layouts/joomla/html/batch to render field, where tooltip has modalTooltip class as handler, and not hasTooltip
Your issue with single article select is not related to batch issue, but... a fix could be done for the issue your report, i think in the layout main for BS modal (could be another PR i could do later... ;-) )

About this PR
This one is not a fix, but additionnal change to a previous PR, to prevent possible future issue.

To Sum-up Batch modal tooltip issue

  • Issue with batch modal tooltip reported here: #9363
  • Fix for the layouts/html/batch/language.php concerning tooltip on first layout field here: #9379
  • And successfull tested and merged in 3.5.0

Information on Bootstrap tooltip

  • Joomla is using hasTooltip as class handler for BS tooltip (previously mootools) (of course, i think you already know that ;-) )
  • hasTooltip class is used in all joomla component views
  • hasTooltip (BS tooltip) container is set by default on body BUT in a modal, this will not show the tooltip, because modal has an higher z-index than the body (known issue of BS modal... again...)
  • in a BS modal, we should set the container for the tooltip on an element included inside the modal. Issue was just this one was set on .modal-body and so were cropped before as it's better to set container to .modal to include the header of the modal window.

Why it's different in Batch modals

  • Batch modals used layouts form fields.
  • In those layouts, the class name for tooltip is not hasTooltip, but modalTooltip (this is good!)
  • in each layout where we use a BS tooltip, as included inside a modal, we need to set the container to .modal

Why this PR?
I did it in language field batch layout, and it was working as the first field loaded in each batch modal. My error was just to not do the same change for access, user and tag (in case one day, language is not the first to be loaded (setting the container), or someone create an override with a different ordering...)

@andrepereiradasilva so, in your testing, you should have the tooltip in batch modals. If you don't have any tooltip on labels hover, there's another issue... Do you have tested on a fresh staging install ?
So, if you don't see any tooltip in batch modal, could you check you have the following script declaration in you page head:

jQuery(document).ready(function(){
    jQuery('.modalTooltip').tooltip({"html": true,"container": ".modal"});
});

And not something else set for .modalTooltip ?...

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

ok, only for batch modals then :)

in your testing, you should have the tooltip in batch modals. If you don't have any tooltip on labels hover, there's another issue... Do you have tested on a fresh staging install ?

I have a clean staging install. tested on windows chrome and firefox and no tooltip in batch modals.

So, if you don't see any tooltip in batch modal, could you check you have the following script declaration in you page head:

Before and after the PR i have:

jQuery(document).ready(function(){
    jQuery('.hasTooltip').tooltip({"html": true,"container": "body"});
});
[...]
jQuery(document).ready(function(){
    jQuery('#filter_search').tooltip({"html": true,"title": "Search title or alias. Prefix with ID: to search for an article ID.","container": "body"});
});
jQuery(document).ready(function(){
    jQuery('.modalTooltip').tooltip({"html": true,"container": ".modal"});
});
avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

i think the problem is related to the "Multilingual Status" link z-index in admin status bar (if i remove the status bar i can see the tooltips!).

See https://github.com/joomla/joomla-cms/blob/staging/administrator/modules/mod_multilangstatus/tmpl/default.php#L15

So the problem clearly doesn't came from your PR

avatar JoomliC
JoomliC - comment - 17 Apr 2016

@andrepereiradasilva You will drive me crazy! ;-)

So, no need to test with this PR, staging shoudl work... then i don't know why it does not work for you as it should (feedback from other users would be welcome especially on windows, as the issue is maybe not related to the batch modal tooltip directly...)

What do you have when you hover labels in batch modals? nothing at all? error in console ?

Could you do a test on 3.5.1 stable about batch modals? (or 3.5.0)

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

@JoomliC please read my previous updated comment

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

Confirmed: Disabled mod_multilanguage admin module and finally i can see the tooltip!!

I guess both of us where right ;)

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

sorry for all of this, but i really couldn't see any tooltip ... enable mod_multilanguage admin module and you'll see what i saw: no tooltip at all.

I will now test your PR with the mod_multilanguage admin module disabled.

avatar JoomliC
JoomliC - comment - 17 Apr 2016

i think the problem is related to the "Multilingual Status" link z-index in admin status bar (if i remove the status bar i can see the tooltips!).

Great!
You've found it!
So, the z-index for module is too hight for the modal....
Now, this would need another PR, to fix it, as the issue is already there with 3.5.1 !

In the same time, maybe the modal z-index should be increase, to be sure the "dark" background is hover all the page contents... What's you opinion ?

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

i think the mod_multilanguage needs to be corrected.

You can see also that with that module enabled the admin status bar is visible!! in all modals.

avatar JoomliC
JoomliC - comment - 17 Apr 2016

@andrepereiradasilva we post in the same time ;-)

The issue you found is not related to this PR, but to a bug already there! (confirmed with 3.5.1)

i think the mod_multilanguage needs to be corrected.

Exactly what i see now too! ;-)
So maybe first to check why this z-index was set to this module ?...

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

So maybe first to check why this z-index was set to this module ?...

because the link opens a modal :)

avatar andrepereiradasilva andrepereiradasilva - test_item - 17 Apr 2016 - Tested successfully
avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

I have tested this item :white_check_mark: successfully on ecd1002


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

avatar JoomliC
JoomliC - comment - 17 Apr 2016

Ok, but miss the reason for this 61eb22d#diff-9c08ca434f4f64bc6d36d803d6f174afR16

Maybe to keep it visible when the modal is open, but.. not sure really needed... (need your lights @dgt41 ;-) )

avatar JoomliC
JoomliC - comment - 17 Apr 2016

@andrepereiradasilva seems that with or without the z-index set in module multi-languages, change nothing in tooltip conflict... and indeed, i see why this one is used.

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

ok. so, we don't know what it is exactly, but it's module multi-languages fault for sure :)

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

i think i found a way to solve the mod_multilangue issue. will make a PR

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

Please test #9958

avatar infograf768
infograf768 - comment - 17 Apr 2016

Hmm
even with @andrepereiradasilva patch, I do not get any tip in the batch modals, also for language.

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

@infograf768 yes that's mod_multilanguage modal code fault.

avatar JoomliC
JoomliC - comment - 17 Apr 2016

@andrepereiradasilva yes, just tested, and fix issue with bottom bar index (better looking!).
ANd yes, unfortunately the issue with tooltip (note: already there in 3.5.1 stable when module multi-languages enabled. note for @infograf768 : not related to this PR, which is the missing change for a PR you already have tested: #9379 ;-) )
So, need to be tested only when module multi-language is disabled, because issue is not the same, and not related to the current change of this PR ;-)

But for sure, a fix to be found for module multi-languages... :-|

avatar joomla-cms-bot
joomla-cms-bot - comment - 17 Apr 2016

This PR has received new commits.

CC: @andrepereiradasilva


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

avatar joomla-cms-bot
joomla-cms-bot - comment - 17 Apr 2016

This PR has received new commits.

CC: @andrepereiradasilva


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

avatar joomla-cms-bot
joomla-cms-bot - comment - 17 Apr 2016

This PR has received new commits.

CC: @andrepereiradasilva


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

avatar joomla-cms-bot
joomla-cms-bot - comment - 17 Apr 2016

This PR has received new commits.

CC: @andrepereiradasilva


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

avatar JoomliC JoomliC - change - 17 Apr 2016
Title
Missing changes of PR #9379 (already merged)
Redo PR #9379 (fix batch tooltip when mod multilanguages enabled)
avatar JoomliC JoomliC - change - 17 Apr 2016
Title
Redo PR #9379 (fix batch tooltip when mod multilanguages enabled)
Redo PR #9379 (fix batch tooltip when mod multilanguage status enabled)
avatar andrepereiradasilva andrepereiradasilva - test_item - 17 Apr 2016 - Tested successfully
avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

I have tested this item :white_check_mark: successfully on 0c1e34a

nice, now the batch modal tooltips work with or without multilanguage status admin module.


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

avatar JoomliC
JoomliC - comment - 17 Apr 2016

@andrepereiradasilva so, i found a way to fix tooltip with module multilanguage status enabled (due to maybe a class conflict...)

So, as all Batch button use "collapseModal" as selector, i have directly set #collapseModal as tooltip container, as in all batch modals, it is the ID of the div modal.
It was working too with container set to div .modal in place of .modal, but i think was better to use directly the unique id of the batch modal.

EDIT: @andrepereiradasilva you really post so fast, i don't have yet time to finish my message ;-)
Thanks for testing, and in the same day, finding and solving another issue! :+1: (in fact 2 issues on mod multilanguage: #9958 (comment) !!!)

@infograf768 the PR is updated, now included a fix too for batch modals tooltip with or without module multilanguage status enabled ;-)

avatar andrepereiradasilva
andrepereiradasilva - comment - 17 Apr 2016

oh, and not related to this PR, but related to tooltips on modals in the croped tooltips in article select ...

image

the problem must be the fact that the tooltip is reaching the iframe boundaries, so IMO we have two options:

  1. not use iframes at all: ie, load the content via ajax and dump it. would also be good for accessibility, but probably a lot of css work
  2. make the tooltip render on bottom instead of top of the labels on this component iframes: don't know if this is simple or not . i guess data-placement="bottom" could be used for that
avatar infograf768 infograf768 - test_item - 18 Apr 2016 - Tested successfully
avatar infograf768
infograf768 - comment - 18 Apr 2016

I have tested this item :white_check_mark: successfully on 0c1e34a

Works fine now.


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

avatar infograf768 infograf768 - change - 18 Apr 2016
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 18 Apr 2016

RTC. Thanks. Can go in 3.5.2


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

avatar joomla-cms-bot joomla-cms-bot - change - 18 Apr 2016
Labels Added: ?
avatar infograf768
infograf768 - comment - 18 Apr 2016

@andrepereiradasilva

I don't have your issue here:

screen shot 2016-04-18 at 11 14 34

avatar rdeutz rdeutz - change - 18 Apr 2016
Milestone Added:
avatar rdeutz rdeutz - change - 18 Apr 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-04-18 09:35:22
Closed_By rdeutz
avatar rdeutz rdeutz - close - 18 Apr 2016
avatar rdeutz rdeutz - merge - 18 Apr 2016
avatar joomla-cms-bot joomla-cms-bot - close - 18 Apr 2016
avatar rdeutz rdeutz - reference | 6898ad8 - 18 Apr 16
avatar rdeutz rdeutz - merge - 18 Apr 2016
avatar rdeutz rdeutz - close - 18 Apr 2016
avatar joomla-cms-bot joomla-cms-bot - change - 18 Apr 2016
Labels Removed: ?
avatar andrepereiradasilva
andrepereiradasilva - comment - 18 Apr 2016

@infograf768 i guess some part depends on browser rendering.
Then go to "Components" -> "Messaging" and click the "My Settings" toolbar button, then try the tooltip in "Local Inbox" label. the issue is the same.

avatar infograf768
infograf768 - comment - 18 Apr 2016

I confirm the issue in that case (Firefox Macintosh)

avatar JoomliC
JoomliC - comment - 18 Apr 2016
the problem must be the fact that the tooltip is reaching the iframe boundaries, so IMO we have two options: not use iframes at all: ie, load the content via ajax and dump it. would also be good for accessibility, but probably a lot of css work make the tooltip render on bottom instead of top of the labels on this component iframes: don't know if this is simple or not . i guess data-placement="bottom" could be used for that

@andrepereiradasilva Yes, iframe issue with BS tooltip.
Another possibility: back using mootools tooltips (joke!) ;-)

Your second option, even if not the best (as definitely an "auto" mode placement is missing in BS2... (but is included in BS3)) is the way to go until iframe are maybe replace little by little by modal body as a layout of the view.
Bottom placement could be set in all view loaded as an iframe.
So, when a little more time, i will see to do a full list of all modals using iframe, in order to check the best way to go for now...
In the same time, if you could list all modals where the issue is today (as on mac, seems that we have less modal with issue than on windows... i think that @infograf768 is on mac too, as i don't have issue with select article, excepted 2 or 3 pixels missing on top of id tooltip ;-) ), i may be able to add a quick valid fix for next version. (but the more i work on BS modals, the more i find many places where some changes could be done to improve all that!).

So, placement bottom for tooltip needed today for:

  • menu item > article type > select article
  • components > messaging > settings
  • ...
avatar JoomliC
JoomliC - comment - 18 Apr 2016

@andrepereiradasilva do you have the issue with versions too (2 last buttons Keep On/Off and Delete) ?
note: i will prepare a PR for known modals using iframe, with tooltip top placement issue, but with other issue too as versions is just because the placement is missing for last 2 buttons ;-)

avatar andrepereiradasilva
andrepereiradasilva - comment - 18 Apr 2016

@JoomliC no not that one because the title is repeated (what?) and with that moves the buttons down so it doesn't reach the iframe boundaries.

image

Update: actually it reaches it by some pixels as you can see in the picture.

avatar JoomliC
JoomliC - comment - 18 Apr 2016

@andrepereiradasilva yes, i saw that about title, and other things too, and will do a PR for a better rendering of Versions html modal ;-)

About tooltip inside iframe, i'm testing a way to fix it without having to edit all related views... But if i don't find the solution, i will do the PR to fix all modal views (which works well indeed with just changing one line!) ;-)

avatar JoomliC
JoomliC - comment - 19 Apr 2016

@andrepereiradasilva and @infograf768, first PR for modal iframe issue: #9991
About com_messages Settings button ;-)

avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Added:
avatar rdeutz rdeutz - change - 1 May 2016
Milestone Removed:

Add a Comment

Login with GitHub to post a comment