Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

The button text not visible #4592

Closed
snetAT opened this issue Apr 12, 2022 · 17 comments · Fixed by #6875
Closed

The button text not visible #4592

snetAT opened this issue Apr 12, 2022 · 17 comments · Fixed by #6875
Labels
bug Something isn't working dark-mode

Comments

@snetAT
Copy link

snetAT commented Apr 12, 2022

Describe the bug
The button text not visible

To Reproduce
Steps to reproduce the behavior:

  1. Activate in Personal - Settings Accessibility - Dark Theme
  2. Set true "Use theme from the integrator" in coolwsd.xml
  3. Open a document and navigate to the view menu for example, and activate "toogle ruler" or other buttons that snap to it.
  4. See error

Expected behavior
The button text should be visible.

Screenshots
Screenshot from 2022-04-10 11-37-06

Client details:

  • OS: Linux, Windows
  • Browser Chrome / Firefox
  • Version 23.03
  • Device: Desktop

Server details

Operating system:
Linux

Web server:
Apache

Database:
MySQL

PHP version:
8.0 fpm

Nextcloud version:
23.03

Version of the richdocuments app
latest

Version of Collabora Online
21.11.3.6

@snetAT snetAT added bug Something isn't working unconfirmed labels Apr 12, 2022
@Andreas-Kainz
Copy link
Contributor

Look like it's a NextCloud Office branding.css issue.

@Ornanovitch
Copy link

I have the same problem with nextcloud dark mode. The bg color is tanken from inline css:

:root {
  ...
  --co-primary-light
  ...
}

But folks from nextcloud office told it was a COOL issue :

@Ezinnem
Copy link
Member

Ezinnem commented Apr 25, 2022

@eszkadev, @pedropintosilva

@timar
Copy link
Member

timar commented Apr 28, 2022

See also #4382

@pedropintosilva
Copy link
Contributor

@snetAT I think this has been fixed, in this example I have set also a dark color from within NC settings(see the active tab label) :
image

@pedropintosilva
Copy link
Contributor

^ tested on master and distro/collabora/co-21-11

@Ornanovitch
Copy link

Ornanovitch commented May 4, 2022

Just upgraded to the latest release (21.11.4.2) and it didn't fix it :/
Is it necessary to clear some cache on the NC level?

@ghost
Copy link

ghost commented Jul 18, 2022

The issue seems to persist on NC 24.0.2 and office 6.1.1 :p
image

Users of my instance also reported missing icons in this Calc menu, which is used to format cell borders. Not sure if this issue is related though, it persists without "Dark Mode" enabled.
image

@Ornanovitch
Copy link

Still an issue on 22.05. How can we progress on the "is it a NC or a CODE issue" question?

@jancborchardt
Copy link

jancborchardt commented Aug 16, 2022

Yep, happening again as mentioned in the meeting. @pedropintosilva @juliushaertl where do we fix it? :)
dark mode header

@juliusknorr
Copy link
Member

So the tricky bit is that we currently read out the css variables which we do only have available for either light or dark theme, so maybe we need to add some extra backend logic to expose the light colors for Collabora. For 25 we could make use of nextcloud/server#33564 for that easily

@Ornanovitch
Copy link

Resolved by @juliushaertl : nextcloud/richdocuments#2550 🥳

@ghost
Copy link

ghost commented Oct 30, 2022

Resolved by @juliushaertl : nextcloud/richdocuments#2550 partying_face

This has not been fully fixed, some icons are still missing
grafik

@Ornanovitch
Copy link

I'm not sure this is the same bug...

2022-10-30-133135_327x219_scrot

@juliusknorr
Copy link
Member

I cannot reproduce this with the latest release, so maybe you can try to upgrade to the latest CODE build and report a separate issue if it still occurs.

Andreas-Kainz added a commit that referenced this issue Jul 10, 2023
Signed-off-by: Andreas-Kainz <andreas_k@abwesend.de>
Change-Id: Ia09efa8530e04f571af4b2e1e7599215746be2df
@Andreas-Kainz
Copy link
Contributor

There are plenty of DarkMode issues (papercuts). For better tracking it would be better to have for each DarkMode papercut a separate issue please.

pedropintosilva pushed a commit that referenced this issue Jul 12, 2023
Signed-off-by: Andreas-Kainz <andreas_k@abwesend.de>
Change-Id: Ia09efa8530e04f571af4b2e1e7599215746be2df
rparth07 pushed a commit to rparth07/online that referenced this issue Jul 12, 2023
Signed-off-by: Andreas-Kainz <andreas_k@abwesend.de>
Change-Id: Ia09efa8530e04f571af4b2e1e7599215746be2df
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working dark-mode
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

9 participants