Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Remove dark-theme hack given upstream fix#94

Draft
jthielen wants to merge 4 commits intoxarray-contrib:main from
jthielen:fix-74
Draft

Remove dark-theme hack given upstream fix #94
jthielen wants to merge 4 commits intoxarray-contrib:main from
jthielen:fix-74

Conversation

@jthielen
Copy link
Collaborator

@jthielen jthielen commented Sep 9, 2022

Change Summary

Simply implements the changes suggested in #74! Did not test locally; hoping that CI will do the trick.

Related issue number

Closes #74.

Checklist

  • (削除) Unit tests for the changes exist (削除ここまで)
  • Tests pass on CI
  • (削除) Documentation reflects the changes where applicable (削除ここまで)

andersy005 reacted with thumbs up emoji
@jthielen jthielen added the documentation Improvements or additions to documentation label Sep 9, 2022
@jthielen jthielen added this to the v0.0.2 milestone Sep 9, 2022
Copy link
Collaborator Author

jthielen commented Sep 9, 2022

This seems to fix the dark theme, but not auto (at least for me in Firefox 104 on Linux) for some reason?

Auto:

image

Light:

image

Dark:

image

I don't notice this issue with the existing hack. Does anyone else have this same issue?

Copy link
Member

I don't notice this issue with the existing hack. Does anyone else have this same issue?

i can confirm that i'm noticing the same issue (on MacOS, Safari)

@jthielen jthielen removed this from the v0.0.2 milestone Sep 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Remove dark-theme hack on next Xarray release

Comments

AltStyle によって変換されたページ (->オリジナル) /