Skip to content
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

Black text on black background in /Settings/FailedRequests #4836

Closed
flyize opened this issue Dec 27, 2022 · 11 comments · Fixed by #4935
Closed

Black text on black background in /Settings/FailedRequests #4836

flyize opened this issue Dec 27, 2022 · 11 comments · Fixed by #4935
Assignees

Comments

@flyize
Copy link

flyize commented Dec 27, 2022

Describe the bug
Black text on black background in /Settings/FailedRequests

To Reproduce
Steps to reproduce the behavior:

  1. Go to /Settings/FailedRequests (I assume with a dark Windows theme)
  2. See error

Expected behavior
Being able to read text ;)

Screenshots
image

Logs (Logs directory where Ombi is located)
If applicable, a snippet of the logs that seems relevant to the bug if present.

Desktop (please complete the following information):

  • OS: Edge on Windows

Ombi Version (please complete the following information):

  • Version 4.33.1
@github-actions
Copy link

Hi!

Thanks for the issue report. Before a real human comes by, please make sure you used our bug report format.

Have you looked at the wiki yet? https://docs.ombi.app/

Before posting make sure you also read our FAQ.

Make the title describe your issue. Having 'not working' or 'I get this bug' for 100 issues, isn't really helpful.

If we need more information or there is some progress we tag the issue or update the tag and keep you updated.

Thanks!

Ombi Bot.

@teambvd
Copy link

teambvd commented Jan 8, 2023

The same thing occurs Firefox, as well as all chromium variants I've tested. One interesting thing I'd found was that enabling the Dark Reader plugin actually seems to clean it up - usually it just inverts the coloration of backgrounds and the like, making sites that are already 'dark' swap to white/bright.

@gold007eye
Copy link

gold007eye commented Mar 9, 2023

Any updates on this. I am seeing the same issue. It looks like maybe just the code for the default font color on this page when "Dark Theme" is used would maybe be the issue.

Desktop (please complete the following information):

OS: Windows 10 | Firefox 110.0.1 (64-bit)

Ombi Version (please complete the following information):
4.35.8

image

@twanariens
Copy link
Collaborator

Try and add this in the Custom CSS box under settings -> configuration -> customization

.table{ color:#FFF; }

@flyize
Copy link
Author

flyize commented Mar 10, 2023

Heh, didn't know we could add custom CSS. Nice!

@gold007eye
Copy link

Try and add this in the Custom CSS box under settings -> configuration -> customization

.table{ color:#FFF; }

Thanks for that information. Will this affect any other Custom CSS that I already have in that Custom CSS section?

@twanariens
Copy link
Collaborator

If you have any custom css related to the table class, then yes

@gold007eye
Copy link

This was the only thing I had in there to disable the Local Login Button:

/* This  will hide the Local Login button */
.login-buttons button:first-of-type { display: none; }

@PalasX
Copy link

PalasX commented Mar 20, 2023

This also affects the newsletter page, making the "submit", "test", "update database", and "trigger now" buttons nearly unreadable.
this can be resolved with
.mat-button-wrapper{ color:#FFF; }

@gold007eye
Copy link

This also affects the newsletter page, making the "submit", "test", "update database", and "trigger now" buttons nearly unreadable. this can be resolved with .mat-button-wrapper{ color:#FFF; }

Thanks for that info. I never used that page so missed it, but added the css you gave just to be safe :)

@gold007eye
Copy link

@tidusjar Would it be possible to included these few changes above hard-coded into a future release by chance? Thanks for all the hard work you do on this project we appreciate it 💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants