qooxdoo / qooxdoo

qooxdoo - Universal JavaScript Framework
http://qooxdoo.org
Other
766 stars 261 forks source link

Wrong colors in IndigoDark for some widgets and states #10517

Closed goldim closed 1 year ago

goldim commented 1 year ago

Looking through Widget Browser I've noticed there are some widget has bad/not suitable colors for theme. Firstly it is a tooltip. There is black color of text and white background. Need to be vice versa. Secondly it is a hovered MenuBar button: both text and background colors during hover are the same (white).

goldim commented 1 year ago

image

sknick commented 1 year ago

I am unable to event get that tab to load. I get the following stack trace:

Widget.js:3297 Uncaught Error: Exception while creating child control 'icon' of widget qx.ui.toolbar.Button[845-0]: Cannot find font in virtual image source: '@MaterialIcons/arrow_back/16'
    at wrapper.__getFont__P_35_20 (Image.js:891:15)
    at wrapper.__setManagedImage__P_35_15 (Image.js:762:27)
    at wrapper._styleSource (Image.js:495:14)
    at wrapper._applySource (Image.js:323:12)
    at wrapper.set (eval at __installFunctionFromCode__P_56_13 (Property.js:1001:36), <anonymous>:3:1252)
    at wrapper.eval [as $$setSourceImpl] (eval at __installFunctionFromCode__P_56_13 (Property.js:1001:36), <anonymous>:6:258)
    at wrapper.eval [as setSource] (eval at __installFunctionFromCode__P_56_13 (Property.js:1001:36), <anonymous>:3:22)
    at wrapper.construct (Image.js:65:12)
    at new wrapper (Class.js:1807:39)
    at wrapper._createChildControlImpl (Atom.js:208:21)
    at wrapper._createChildControl (Widget.js:3282:30)
    at wrapper.getChildControl (Widget.js:3186:19)
    at wrapper._showChildControl (Widget.js:3196:26)
    at wrapper._handleIcon (Atom.js:247:14)
    at wrapper._applyIcon (Atom.js:276:12)
    at wrapper.set (eval at __installFunctionFromCode__P_56_13 (Property.js:1001:36), <anonymous>:3:1189)
goldim commented 1 year ago

Did you get last changes from master? There was PR related to fix some things about IndigoDark theme.

sknick commented 1 year ago

Yes, I'm using latest. I had this issue before too.

sknick commented 1 year ago

Perhaps this has something to do with bootstrapping the compiler and installing via npm install --save <local qooxdoo>? I really have no idea.

sknick commented 1 year ago

I guess I should elaborate and say that this issue isn't specific to the new theme... I've never been able to get that tab to load in the widget browser, even prior to the new theme.

sknick commented 1 year ago

@goldim How are you building the widget browser? Perhaps the way in which you're building the framework itself is different and is why you avoid the error I'm seeing?

goldim commented 1 year ago

@sknick I used some old widget browser from my fork project (theme). Just added IndigoDark theme there and checked how it works. I will try to add Widget Browser as dependency into new theme project. Probably instead of building widget browser as separate project install it as package from qooxdoo package system?

goldim commented 1 year ago

@sknick Yes, there is an error which you've described in toolbar/menu. I used qooxdoo compiler/framework with current changes.

  1. ..\qooxdoo\bootstrap\qx create test --type package --out . -I
  2. ..\qooxdoo\bootstrap\qx package install qooxdoo/qxl.widgetbrowser
  3. ..\qooxdoo\bootstrap\qx package install qooxdoo/qxl.versionlabel
  4. ..\qooxdoo\bootstrap\qx package install qooxdoo/qxl.formdemo
  5. npx qx serve -S
sknick commented 1 year ago

So I followed your steps above but I'm still getting the same stack trace.

sknick commented 1 year ago

I'm seeing this when I compile Qooxdoo after installing the packages you specified:

Assets required for webfont MaterialIconsOutlined are not available in application widgetbrowser, consider using @asset to include qx/iconfont/MaterialIcons/materialiconsoutlined-v101.eot,qx/iconfont/MaterialIcons/materialiconsoutlined-v101.woff2,qx/iconfont/MaterialIcons/materialiconsoutlined-v101.woff,qx/iconfont/MaterialIcons/materialiconsoutlined-v101.otf
goldim commented 1 year ago

I've created an issue about Material Icons in qxl.widgetbrowser but there is compiled version which runs on github pages.