-
-
Notifications
You must be signed in to change notification settings - Fork 436
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Lots of warning in the browser console related to swatches #4217
Comments
@empiricompany maybe you can review this? |
I look this fast cause I am on mobile and seems by your screenshots that you didn't load all the css and javascript correctly |
I checked, and on Firefox, it doesn't give any warnings, while on Chrome it does. They can be safely ignored considering the benefit they bring, as the warnings do not compromise functionality. This is how it should look: We spent a lot of time finding a solution for both Windows and Mac with some compromises to ensure maximum backward compatibility, and to be completely honest, I have no desire to go back and work on the code again. This was the PR where also you @addison74 have followed: #3686 |
I appreciate you checking this report. Indeed I followed it until a certain moment when I gave up for some reasons. The problem is visible in Chromium, so in Chrome, Edge, Brave. Maybe we can all find a solution because this is the only section where I found warnings in the console. Regarding that X, the legacy theme from OpenMage must be taken into account. I say it as a personal opinion and if someone is interested in doing it, it would be a useful change. For this purpose I would reopen the report to be seen by others. Thank you for your understanding. |
@addison74 make sure to refresh browser cache. Backend looks fine for me (Firefox). (If it does not work with Chrome, it should be checked) I'd care about the console warnings - if there are any. (not tested yet) |
@sreichel - I always use browsers with cache disabled. Before reporting this issue, related to Chromium flavors, I cleared the cache to be sure. |
Confirmed. @empiricompany the broken "X" images are in legacy theme, not in the new openmage. Console warnings are only in Chrome, not in Firefox. |
Do we still need to support the legacy theme? Use the official OpenMage backend theme by default and there are no problems. |
As long we have that theme, it should be supported. Working on it. |
As of now I think it is legitimate to want to fix bugs in the legacy theme as long as it exists in the main repo. However, there is a good case to be made for dropping support of it. To that end, I have opened a discussion here: #4230 |
Go to Backend > Attributes > Manage Attributes and edit the Color attribute. Go to the Manage Label / Options tab on the left side. With the sample pack there are 19 options, from Black to Yellow.
Open the browser console. You will see 19 warning related to the value "false" used in the swatches. The screenshot is bellow
As a personal opinion, since I did not follow the PR that made this change, it would have been preferable for that ugly X to be on a new column, next to the swatch or on the last column. This is how the change would have looked for the last column. Even a
Even without text and it would have looked better compared to the other elements on the page
The text was updated successfully, but these errors were encountered: