-
Notifications
You must be signed in to change notification settings - Fork 61
refactor(issue-367): clean up flask ui for public preview #371
Conversation
color: #505050; | ||
font-family: Roboto, sans-serif; | ||
font-display: swap; | ||
font-family: 'Google Sans', sans-serif !important; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to avoid !important
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not ideal for now, but would like to still apply it for now. Added this in order to make sure the donations table picked this up. I wanted to avoid adding too many css lines to make sure the font was being applied.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we leave a TODO
comment linking to a GitHub issue to remove the important
?
(If and when we deprecate the Flask frontend, we can close that issue + remove the TODO.)
|
||
.mdc-top-app-bar, | ||
.mdc-button { | ||
background: #306DFD !important; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ditto - can we avoid !important
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As above - keeping !important
sounds fine, provided we include a link (via code comments) to a GitHub issue tracking its (eventual) removal.
@ace-n All updated! Ready for re-review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM provided we document removing !important
as a TODO.
color: #505050; | ||
font-family: Roboto, sans-serif; | ||
font-display: swap; | ||
font-family: 'Google Sans', sans-serif !important; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we leave a TODO
comment linking to a GitHub issue to remove the important
?
(If and when we deprecate the Flask frontend, we can close that issue + remove the TODO.)
|
||
.mdc-top-app-bar, | ||
.mdc-button { | ||
background: #306DFD !important; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As above - keeping !important
sounds fine, provided we include a link (via code comments) to a GitHub issue tracking its (eventual) removal.
Issue: #367
Cleaning up flask frontend for public preview.
Resolves the following views:
before
after
before
after
before
after
before
after
before
after