-
Notifications
You must be signed in to change notification settings - Fork 243
Change color for text in HTML report #116
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
Comments
@APshenkin thank you for pointing this out. Let me spend some time investigating options. |
@davehunt what's your preference.
|
@m8ttyB let's tweak the contrast of the current CSS to improve accessibility by default. I think having an option to provide a CSS file as an override would be neat, but it's not clear if there's a thirst for this. @APshenkin do you have any thoughts? |
@APshenkin An additional question to go along with @davehunt's It's also possible to increase the font-size, although would using the built-in ability of your browser to increase font-size work in this instance? |
@APshenkin apologies, this thread fell of my radar. Let me take a look at this again early next week and scope a solution. |
@davehunt what's your preference, tweaking the existing CSS (defining the write ratios could take awhile), or allowing users to insert their own CSS styling? |
@m8ttyB I like the idea of having a |
I would also propose that the theme is always set as a base style, and the user specified CSS simply allows selected styles to be replaced. That way, the two features would be able to work together. |
Are you still interested to work on this @m8ttyB or should I unassign in case someone else can take it? |
To throw my two cents into the discussion, I vote for user-supplied CSS rather than a change to the core styles. This will provide a package-based solution that can be individually tailored as required. Much of my current test output is unreadable: I use |
Thanks for your thoughts @i-am-david-fernandez, I think the best way forward is allowing a user to specify a |
@davehunt I've submitted a PR for my attempt at implementing this. Hopefully I've accurately captured the needs and desires as described here. |
Hi!
First, thank you for this plugin!
I want to recommend to change font color in report because read grey text on white background is "hard" for eyes. Also if we can increase font size it would be great!
Thanks in advance!
The text was updated successfully, but these errors were encountered: