Fossil SCM

Avoided a divide by 1000 by changing a value formerly specified in floating point seconds to be specified in milliseconds, so we don't need the arithmetic.

wyoung 2018-09-12 12:33 trunk
Commit 1d5eea6669af52b3784de69293e05e8d99c520ad2fa70ae25db15c8c25c4cfd8
1 file changed +2 -2
--- skins/default/js.txt
+++ skins/default/js.txt
@@ -96,12 +96,12 @@
9696
if (animate) {
9797
// Set up a CSS transition to animate the panel open and
9898
// closed. Only needs to be done once per page load.
9999
// Based on https://stackoverflow.com/a/29047447/142454
100100
calculatePanelHeight();
101
- panel.style.transition = 'max-height ' +
102
- (animMS / 1000) + 's ease-in-out';
101
+ panel.style.transition = 'max-height ' + animMS +
102
+ 'ms ease-in-out';
103103
panel.style.overflowY = 'hidden';
104104
panel.style.maxHeight = '0';
105105
showPanel();
106106
}
107107
panel.style.display = 'block';
108108
--- skins/default/js.txt
+++ skins/default/js.txt
@@ -96,12 +96,12 @@
96 if (animate) {
97 // Set up a CSS transition to animate the panel open and
98 // closed. Only needs to be done once per page load.
99 // Based on https://stackoverflow.com/a/29047447/142454
100 calculatePanelHeight();
101 panel.style.transition = 'max-height ' +
102 (animMS / 1000) + 's ease-in-out';
103 panel.style.overflowY = 'hidden';
104 panel.style.maxHeight = '0';
105 showPanel();
106 }
107 panel.style.display = 'block';
108
--- skins/default/js.txt
+++ skins/default/js.txt
@@ -96,12 +96,12 @@
96 if (animate) {
97 // Set up a CSS transition to animate the panel open and
98 // closed. Only needs to be done once per page load.
99 // Based on https://stackoverflow.com/a/29047447/142454
100 calculatePanelHeight();
101 panel.style.transition = 'max-height ' + animMS +
102 'ms ease-in-out';
103 panel.style.overflowY = 'hidden';
104 panel.style.maxHeight = '0';
105 showPanel();
106 }
107 panel.style.display = 'block';
108

Keyboard Shortcuts

Open search /
Next entry (timeline) j
Previous entry (timeline) k
Open focused entry Enter
Show this help ?
Toggle theme Top nav button