Skip to content

fix: Set font size from cookie before DOM is ready, to avoid flickering#4882

Merged
rjsparks merged 4 commits into
ietf-tools:mainfrom
larseggert:fix-flickering
Dec 15, 2022
Merged

fix: Set font size from cookie before DOM is ready, to avoid flickering#4882
rjsparks merged 4 commits into
ietf-tools:mainfrom
larseggert:fix-flickering

Conversation

@larseggert

Copy link
Copy Markdown
Collaborator

No description provided.

@rjsparks rjsparks requested a review from NGPixel December 13, 2022 14:35
@codecov

codecov Bot commented Dec 13, 2022

Copy link
Copy Markdown

Codecov Report

Merging #4882 (df1d1cb) into main (bc1cba1) will increase coverage by 0.08%.
The diff coverage is n/a.

❗ Current head df1d1cb differs from pull request most recent head 0a599a5. Consider uploading reports for the commit 0a599a5 to get more accurate results

@@            Coverage Diff             @@
##             main    #4882      +/-   ##
==========================================
+ Coverage   88.47%   88.55%   +0.08%     
==========================================
  Files         296      296              
  Lines       39795    40076     +281     
==========================================
+ Hits        35208    35490     +282     
+ Misses       4587     4586       -1     
Impacted Files Coverage Δ
ietf/doc/views_search.py 89.85% <0.00%> (+0.20%) ⬆️
ietf/utils/mail.py 79.85% <0.00%> (+0.23%) ⬆️
ietf/person/utils.py 86.98% <0.00%> (+0.59%) ⬆️
ietf/doc/utils.py 91.21% <0.00%> (+3.00%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Comment thread ietf/static/js/document_html.js Outdated
@larseggert larseggert requested a review from NGPixel December 14, 2022 09:45
@rjsparks rjsparks merged commit 8a599b7 into ietf-tools:main Dec 15, 2022
@larseggert larseggert deleted the fix-flickering branch December 15, 2022 15:40
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Dec 19, 2022
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.

3 participants