[#4507] - JHtmlBootstrap::tooltip() hide vs. hideme
- Closed
- 25 Oct 2015
- Medium
- Build: staging
- # 4507
- Diff
- itbra:patch-28
- Success Human Test Results: 3 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
It appears the hide event is not fired when triggering an element to show the tooltip. However, the hideme event is fired. This can easily be tested with the following code:
JHtml::_('bootstrap.tooltip', '.hasStickyTooltip', array(
'onShow' => 'function ( e ) {'
.'alert("onShow executed")'.
'}',
'onShown' => 'function ( e ) {'
.'alert("onShown executed")'.
'}',
'onHide' => 'function ( e ) {'
.'alert("onHide executed")'.
'}',
'onHideMe' => 'function ( e ) {'
.'alert("onHideMe executed")'.
'}',
'onHidden' => 'function ( e ) {'
.'alert("onHidden executed")'.
'}'
));
This proves the message onHide executed never appears. To get hands on the event fired before the widget is getting hidden i added the corresponding listener onHideMe to the configuration array. With this listener in place the message onHideMe executed is shown and the corresponding event is handled by the plugin.
jissues-bot
- | Labels |
Added:
?
|
||
Huh? Why does the build appear to failed when is was successful yesterday?
So I'd say that it's not possible
I'm afraid, i don't get what you mean.
Don't worry about Travis. It seems to be ok.
brianteeman
- | Category | ⇒ | Libraries |
tflm84
- 
I have tested this item 
successfully on a31c5b9
Before Patch:
onHidden executed
After Patch:
onHide executed
onHidden executed
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4507.
I have tested this item successfully on a31c5b9
Works as expected!
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4507.
I have tested this item successfully on a31c5b9
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4507.
Yep works as @matrikular describes it.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4507.
| Milestone | |||
| Milestone |
Added: |
||
| Status | Pending | ⇒ | Ready to Commit |
joomla-cms-bot
- | Labels |
Added:
?
|
||
rdeutz
- | Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-10-25 09:59:53 |
| Closed_By | ⇒ | rdeutz |
| Labels |
Removed:
?
|
||
| Milestone | |||
| Milestone |
Added: |
||
| Milestone |
Added: |
||
| Milestone | |||
There was a conflict between bootstrap hide event and mootools hide even. To keep backward compatibility with Mootools we were forced to rename the event.
I think we could just replace the existing hide to map it to the hideme bootstrap event?
So I'd say that it's not possible