GUACAMOLE-346: Remove "paused" notification. Correct UI behavior.

This commit is contained in:
Michael Jumper
2017-07-15 17:45:39 -07:00
parent 387302db48
commit 929452e701
3 changed files with 8 additions and 13 deletions

View File

@@ -28,7 +28,7 @@
<body> <body>
<!-- Guacamole recording player --> <!-- Guacamole recording player -->
<div id="player" class="paused"> <div id="player">
<!-- Player display --> <!-- Player display -->
<div id="display"> <div id="display">
@@ -39,9 +39,6 @@
<button id="cancel-seek">Cancel</button> <button id="cancel-seek">Cancel</button>
</p> </p>
</div> </div>
<div class="paused-notification">
<p>Paused.</p>
</div>
</div> </div>
</div> </div>

View File

@@ -34,7 +34,6 @@
bottom: 0; bottom: 0;
} }
#player .paused-notification,
#player .seek-notification { #player .seek-notification {
color: white; color: white;
@@ -50,11 +49,6 @@
display: table; display: table;
} }
#player.paused .paused-notification {
display: table;
}
#player .paused-notification p,
#player .seek-notification p { #player .seek-notification p {
display: table-cell; display: table-cell;
text-align: center; text-align: center;

View File

@@ -162,13 +162,11 @@
// If playing, the play/pause button should read "Pause" // If playing, the play/pause button should read "Pause"
recording.onplay = function() { recording.onplay = function() {
playPause.textContent = 'Pause'; playPause.textContent = 'Pause';
player.className = 'playing';
}; };
// If paused, the play/pause button should read "Play" // If paused, the play/pause button should read "Play"
recording.onpause = function() { recording.onpause = function() {
playPause.textContent = 'Play'; playPause.textContent = 'Play';
player.className = 'paused';
}; };
// Toggle play/pause when display or button are clicked // Toggle play/pause when display or button are clicked
@@ -182,6 +180,7 @@
// Resume playback when cancel button is clicked // Resume playback when cancel button is clicked
cancelSeek.onclick = function cancelSeekOperation(e) { cancelSeek.onclick = function cancelSeekOperation(e) {
recording.play(); recording.play();
player.className = '';
e.stopPropagation(); e.stopPropagation();
}; };
@@ -213,7 +212,12 @@
positionSlider.onchange = function sliderPositionChanged() { positionSlider.onchange = function sliderPositionChanged() {
// Request seek // Request seek
recording.seek(positionSlider.value); recording.seek(positionSlider.value, function seekComplete() {
// Seek has completed
player.className = '';
});
// Seek is in progress // Seek is in progress
player.className = 'seeking'; player.className = 'seeking';