-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
Auditorium UI: Emphasise weekends better #480
Comments
Can I work on it ? If so please assign it to me. |
@haridevelops let me know if there are any questions |
@hendr-ik Should I take this, seems that no progress further by other person. |
Thanks @prijeshb , let's maybe check with @haridevelops if they still plan to work on this? Any input from your side? |
Also, looked documentation but didn't found any relevant attribute to set background color for ticks |
@prijeshb yeah, this will require some creative use of Plotly (negative bars maybe?) and is not something that is supported out of the box as-is. Maybe there is some other solution we don't know of yet? Let me know if you want to work on this in any case so I can assign you. Thanks. |
@hendr-ik |
@haridevelops When I try applying a offen/auditorium/src/views/components/auditorium/chart.js Lines 60 to 62 in 0565713
it doesn't really display any background for me. Is there something needed in addition to that? |
@m90 I have locally up the code. I would like to recommend different aspect here emphasising weekend. Thoughts? |
@haridevelops Thanks for doing that research. It's tricky business indeed.
Thing is we need to / want to comply with WACG 2.1 Accessibility Guidelines here, so styling this accordingly is a complicated topic. @hendr-ik is on holidays for a few more days, but I would suggest he follows up on this with you once he's back. Does this sound like a plan? |
perfect @m90 |
@haridevelops Thanks again for your efforts and the suggestion. Unfortunately, we cannot use green for highlighting because it is only used in connection with links in our UI. I think we need to dig deeper into Plotly and create a design that fits the possibilities. |
@haridevelops Please do not hesitate to submit your recommendation as a pull request in order to be officially recognized. Unfortunately, for the reasons mentioned above, we will not be able to merge it. |
This is the way we currently emphasise weekends in our Plotly bar charts - simple numbers, only slightly larger, italicized:
![bars is](https://camo.githubusercontent.com/901e7f4c0e496e01819d500a67a809c7509c352eee5c83f58aa12e6e4802a496/68747470733a2f2f6f6666656e2e6769746875622e696f2f70726573732d6b69742f6f6666656e2d6861636b746f626572666573742f6f6666656e5f626172735f69732e706e67)
We consider this not obvious enough and woud like to update it this way:
![bars want](https://camo.githubusercontent.com/793eacb2e159c8bb47e50c73d86d7a3ab54f34fcb83d92c520ad80710f037138/68747470733a2f2f6f6666656e2e6769746875622e696f2f70726573732d6b69742f6f6666656e2d6861636b746f626572666573742f6f6666656e5f626172735f77616e742e706e67)
Ideally it should be implemented directly in Plotly without extra css. The relevant component is this one:
https://github.com/offen/offen/blob/master/auditorium/src/views/components/auditorium/chart.js
The text was updated successfully, but these errors were encountered: