GitHub/noVNC
1
0
Fork 0
mirror of https://github.com/novnc/noVNC.git synced 2026-05-26 07:08:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove redundant mobile buttons grouping

Browse Source 
There's only one button in the group, and the wrapping div messed with
the flex box layout
This commit is contained in:
Tobias
2026-02-18 12:43:59 +01:00
parent 44343556e0
commit 052242df5d
2 changed files with 7 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
app/styles/base.css
View File

@@ -718,16 +718,15 @@ html {
}
/* noVNC Touch Device only buttons */
:root:not(.noVNC_connected) #noVNC_mobile_buttons {
:root:not(.noVNC_connected) #noVNC_keyboard_button {
display: none;
}
@media not all and (any-pointer: coarse) {
/* FIXME: The button for the virtual keyboard is the only button in this
group of "mobile buttons". It is bad to assume that no touch
devices have physical keyboards available. Hopefully we can get
a media query for this:
/* FIXME: It is bad to assume that no touch devices have physical
keyboards available. Hopefully we can get a media query
for this:
https://github.com/w3c/csswg-drafts/issues/3871 */
:root.noVNC_connected #noVNC_mobile_buttons {
:root.noVNC_connected #noVNC_keyboard_button {
display: none;
}
}

2
vnc.html
View File

@@ -127,10 +127,8 @@
title="Move/Drag viewport">
<!--noVNC touch device only buttons-->
<div id="noVNC_mobile_buttons">
<input type="image" alt="Keyboard" src="app/images/keyboard.svg"
id="noVNC_keyboard_button" class="noVNC_button" title="Show keyboard">
</div>
<!-- Extra manual keys -->
<input type="image" alt="Extra keys" src="app/images/toggleextrakeys.svg"