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

feat: Upgrade bootstrap to 5.3.0 and enable dark mode #5509

Merged
merged 143 commits into from
Jun 13, 2023

Conversation

larseggert
Copy link
Collaborator

Fixes #3365

@codecov
Copy link

codecov bot commented Apr 25, 2023

Codecov Report

❗ No coverage uploaded for pull request base (feat/dark-mode@1ec403d). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 685a20f differs from pull request most recent head 7497143. Consider uploading reports for the commit 7497143 to get more accurate results

@@                Coverage Diff                @@
##             feat/dark-mode    #5509   +/-   ##
=================================================
  Coverage                  ?   88.74%           
=================================================
  Files                     ?      285           
  Lines                     ?    39853           
  Branches                  ?        0           
=================================================
  Hits                      ?    35366           
  Misses                    ?     4487           
  Partials                  ?        0           

Try again with Django 3.2
Patch no longer applies. Needed until Django 4.2.1, so bring it back
if we deploy something earlier than that to production.
Relying on Meta.ordering to order querysets involving GROUP BY queries
is deprecated and will be dropped in Django 3.1.
Not sure why, but if I change the DeprecationWarning filter to
"error", I get a SyntaxError from the \[ because it is an invalid
escape sequence. Not sure why that change triggers it, but
"\[" and "\\[" are the same, so this is a no-op.
chore: merge main into feat/django4
* test: Remove ScheduleEditTests.testUnschedule

Has been disabled under Django 2. Simple refactoring does not make it
functional under Django 3. Probably because we know that Selenium does
not handle HTML5 drag-and-drop well. Discarding until we move to a
better JS testing framework.

* test: Remove unused imports
…ls#5726)

* fix: Return NomComs, not Groups, from active_nomcoms filter

* fix: deduplicate and order nomcoms in active_nomcoms filter
@larseggert
Copy link
Collaborator Author

* The IETF logo on the homepage should vary to light text when in dark mode.

I made that change (also to other logos).

* The agenda page isn't adapted for dark mode (I'll contribute to this PR to add the missing bits)

Thanks!

* Might be best to add a button in the header or footer to quickly switch between light and dark mode. Not everyone will know to look in Preferences.

Maybe. It'd make it pretty busy.

When first loading the site, not logged in, it was automatically set to "Dark" instead of "Auto"?

Odd. Do you maybe have previous data in localstorage? I just tried in private browsing mode and it comes up in "auto".

rjsparks and others added 24 commits June 1, 2023 09:32
…etf-tools#5730)

* fix: Accept a Path as source for a PlaintextDraft

* fix: Guard against failure to extract PlaintextDraft title

* fix: Ignore failure to extract text draft title unless it is needed
* fix: Extract document creation date from XML draft

* test: Fix test
chore: Merge main into feat/django4
chore: Use permissive cross-origin-opener-policy setting
@rjsparks rjsparks merged commit ce3114c into ietf-tools:feat/dark-mode Jun 13, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 17, 2023
@larseggert larseggert deleted the feat-dark-mode branch August 10, 2023 15:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Datatracker dark mode
5 participants