From 81f34e224c0e28ebc37ecd9bd7ea7e607fb98486 Mon Sep 17 00:00:00 2001 From: Abijeet Date: Sun, 3 Dec 2017 18:26:54 +0530 Subject: Allows adding of hex color to the theme-color Fixes #7158. Adds a # on the color if missing. Increased maxlength, added hash:true for jscolor, and adding a # if not present on the change event. Since the input element now allows a hex code, changed values to hexcode. In addition, added a function to get RGB array from hex or rgb values. Calling it in both methods and using it to perform comparison. Also changed the way we were determining whether the jscolor component had loaded. Changed the control to use data-jscolor rather than defining opts in the class. Signed-off-by: Abijeet --- apps/theming/js/settings-admin.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) (limited to 'apps/theming/js') diff --git a/apps/theming/js/settings-admin.js b/apps/theming/js/settings-admin.js index 44a799a19b4..7df1bbf1125 100644 --- a/apps/theming/js/settings-admin.js +++ b/apps/theming/js/settings-admin.js @@ -206,7 +206,11 @@ $(document).ready(function () { }); $('#theming-color').change(function (e) { - setThemingValue('color', '#' + $(this).val()); + var color = $(this).val(); + if (color.indexOf('#') !== 0) { + color = '#' + color; + } + setThemingValue('color', color); }); $('.theme-undo').click(function (e) { -- cgit v1.2.3