User tests: Successful: Unsuccessful:
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
Status | New | ⇒ | Pending |
Labels |
Added:
?
|
Category | ⇒ | Layout |
@andrepereiradasilva just tested with latest staging and fresh install on Chrome (mac), and i have the label tooltip:
:-|
Note: with or without patch, tooltip works for me...
ok, so clearly is something on my part.
@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
Information on Bootstrap tooltip
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...).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
hasTooltip
, but modalTooltip
(this is good!).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
?...
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"});
});
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!).
So the problem clearly doesn't came from your PR
@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)
Confirmed: Disabled mod_multilanguage admin module and finally i can see the tooltip!!
I guess both of us where right ;)
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.
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 ?
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.
@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 ?...
So maybe first to check why this z-index was set to this module ?...
because the link opens a modal :)
I have tested this item successfully on ecd1002
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 ;-) )
@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.
ok. so, we don't know what it is exactly, but it's module multi-languages fault for sure :)
i think i found a way to solve the mod_multilangue issue. will make a PR
Hmm
even with @andrepereiradasilva patch, I do not get any tip in the batch modals, also for language.
@infograf768 yes that's mod_multilanguage modal code fault.
@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... :-|
This PR has received new commits.
This PR has received new commits.
This PR has received new commits.
This PR has received new commits.
Title |
|
Title |
|
I have tested this item successfully on 0c1e34a
nice, now the batch modal tooltips work with or without multilanguage status admin module.
@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! (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 ;-)
oh, and not related to this PR, but related to tooltips on modals in the croped tooltips in article select ...
the problem must be the fact that the tooltip is reaching the iframe boundaries, so IMO we have two options:
data-placement="bottom"
could be used for thatI have tested this item successfully on 0c1e34a
Works fine now.
Status | Pending | ⇒ | Ready to Commit |
RTC. Thanks. Can go in 3.5.2
Labels |
Added:
?
|
Milestone |
Added: |
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 |
Labels |
Removed:
?
|
@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.
I confirm the issue in that case (Firefox Macintosh)
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:
@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 ;-)
@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!) ;-)
@andrepereiradasilva and @infograf768, first PR for modal iframe issue: #9991
About com_messages Settings button ;-)
Milestone |
Removed: |
Milestone |
Added: |
Milestone |
Added: |
Milestone |
Removed: |
hum ... for me the tooltip doesn't exist with or without the PR (using latest staging on chrome).