? Pending

User tests: Successful: Unsuccessful:

avatar 810
810
5 Jan 2020

When you enable dark mode (flag setting) on the browser, all websites are showed as dark.

But on the card the color was to white and wasn't showed correctly.

Summary of Changes

card border color

Testing Instructions

edge://flags
chrome://flags

enable Force Dark Mode for Web Contents

Expected result

after

Actual result

before

avatar 810 810 - open - 5 Jan 2020
avatar 810 810 - change - 5 Jan 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 Jan 2020
Category Front End Templates (site)
avatar brianteeman
brianteeman - comment - 5 Jan 2020

just going from your screenshot

If the card outline needs to be changed then so does the input border as seen in the login module and many more places.

I am not against updating to make it compatible with the dark mode flag but its better to do it all at once in a consistent manner and not piecemeal

avatar 810
810 - comment - 5 Jan 2020

Many things are getting changed by the browser (the feature itself), only we need to change some conflicts. I didn't found any other issue, then the top logo banner. But i need to check more conflicts.

Like this issue on github:
image

avatar 810 810 - change - 27 Jan 2020
Labels Added: ?
avatar 810
810 - comment - 27 Jan 2020

i don't see any other issues, so we can merge this one.

avatar jwaisner jwaisner - change - 6 Mar 2020
The description was changed
Priority Medium Low
avatar joomla-cms-bot joomla-cms-bot - edited - 6 Mar 2020
avatar david0296 david0296 - test_item - 3 Aug 2020 - Tested unsuccessfully
avatar david0296
david0296 - comment - 3 Aug 2020

I have tested this item ? unsuccessfully on 72f7f0a

The white borders still exist.
I was using Microsoft Edge Version 84.0.522.52


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27406.

avatar jmeintrup jmeintrup - test_item - 4 Aug 2020 - Tested successfully
avatar jmeintrup
jmeintrup - comment - 4 Aug 2020

I have tested this item successfully on 72f7f0a

Works as expected, but your testing instructions are incomplete.

Before testing the change you need to run npm run build:css or the changes won't take place (obviously).


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27406.

avatar johndeutesfeld johndeutesfeld - test_item - 4 Aug 2020 - Tested successfully
avatar johndeutesfeld
johndeutesfeld - comment - 4 Aug 2020

I have tested this item successfully on 72f7f0a

Works as expected on the frontend but changes did not apply in the backend.

as mentioned before, you have to run npm run build:css before testing else it won't show any differences.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27406.

avatar roland-d
roland-d - comment - 4 Aug 2020

@david0296 Did you run the npm run build:css ?

avatar david0296
david0296 - comment - 4 Aug 2020

@roland-d I tried again with npm run build:css and it worked

avatar roland-d roland-d - alter_testresult - 4 Aug 2020 - david0296: Tested successfully
avatar roland-d roland-d - change - 4 Aug 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-08-04 14:05:22
Closed_By roland-d
avatar roland-d roland-d - close - 4 Aug 2020
avatar roland-d roland-d - merge - 4 Aug 2020
avatar joomla-cms-bot joomla-cms-bot - change - 4 Aug 2020
Category Front End Templates (site) Front End Templates (site) NPM Change

Add a Comment

Login with GitHub to post a comment