Sometimes I experience that the sidebar animation gets stuck on hover.
Please see the attached video:
https://www.dropbox.com/s/48qlqkmw2fij7kh/sidebar.mov?dl=0
Labels |
Added:
?
|
Labels |
Added:
?
|
I'm not able to reproduce this on beta2.
Are you able to have the same issue on beta2 or only on staging ?
I'm gonna test in some different browsers later tonight. This issue exist both in staging and in beta2
Category | ⇒ | Templates (admin) UI/UX |
Looked deeper into the problem, it origins from the collapse class in Isis index.php on line 204
<div class="subhead-collapse collapse">
I would recommend removing the class, as it seems to only effect Safari
@losedk I was not able to reproduce exactly what you have on Safari 8, but i have a wrong animation of the sidebar (similar to your issue).
And it's true that if we remove the collapse, it's working fine :
<div class="subhead-collapse">
So, i wonder what for this "collapse" is, and if we could remove it... @dgt41 what's your opinion about this ? Should we create a PR to remove the "collapse" in subhead-collapse ?
This collapse I think is the way to represent the toolbar as accordion for mobiles. If we remove it then the toolbar buttons will always be visible. Let me look at it later on when I will be at my desk...
Yeah, if removing the class the toolbar is always present
@losedk One more try:
with #6696 open template.js and paste:
if (window.isisStickyToolbar == 1) {
processScrollInit();
processScroll();
$(window).on('resize', processScrollInit);
$(window).on('scroll', processScroll);
}
after line 206 (after the } press return and the paste this snippet)
and after line 180 (same)
It should work, as it will re initiate the toolbar/header every time that the sidebar is evaluated!
@losedk in current staging if you remove the last <script>
in index.php of isis, you should not get any problems, therefore this is the code that messes up the display. Sometimes it fires to early, (I think) and the final result is a mess…
Maybe we should re write that little thing there
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-06-24 17:51:15 |
Closed_By | ⇒ | losedk | |
Build | master | ⇒ | staging |
Confirmed, I ve seen this as well