[pve-devel] [PATCH pve-manager 1/3] css improvements for crisp theme

Emmanuel Kasper e.kasper at proxmox.com
Mon Mar 14 13:35:09 CET 2016


* usability improvement for enabled buttons:
in the default theme, ExtJS uses two different nuances of grey to
distinguish enabled or disabled buttons
the problem is that compared to the full black of the panels titles, it gives
the impression that everything is disabled (the contrast is not strong
enough between the two grey nuances)
the fix is to replace the dark grey of an 'enabled' button with black, similar with
what we had in ExtJS and the Classic theme

* re add menu header
this was lost when moving to crisp theme

reasoning and fix identic to

160b249a8386607da3eb50b3b4871d28050a9795
by default the title in extjs6 menus have no style
this patch uses the same style as the tab headers
---
 www/css/ext6-pve.css | 16 +++++++++++++++-
 1 file changed, 15 insertions(+), 1 deletion(-)

diff --git a/www/css/ext6-pve.css b/www/css/ext6-pve.css
index 3c3684c..6f5af32 100644
--- a/www/css/ext6-pve.css
+++ b/www/css/ext6-pve.css
@@ -1 +1,15 @@
-/* empty for now */
+/* overwrite to use full black for enabled buttons */
+.x-btn-inner-default-toolbar-small {
+    font: 300 12px/16px helvetica, arial, verdana, sans-serif;
+    color: #000;
+    padding: 0 5px;
+    max-width: 100%;
+}
+
+/* add missing class for context menu header */
+.x-menu-header {
+    font: 400 13px/20px 'Open Sans', 'Helvetica Neue', helvetica, arial, verdana, sans-serif;
+    color: #fff;
+    padding: 4px;
+    background-color: #3892d4;
+}
\ No newline at end of file
-- 
2.1.4





More information about the pve-devel mailing list