-
Notifications
You must be signed in to change notification settings - Fork 5
Redesign/general-improvements-9 #1468
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
Conversation
Fixes a bug with accordion context being stale on non-question/concept/generic pages
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.
nice, one small visual bug but otherwise this all looks good 👍
@@ -190,12 +190,15 @@ export const HomepagePhy = () => { | |||
</div> | |||
<ShowLoadingQuery | |||
query={eventsQuery} | |||
defaultErrorTitle={"Error loading events list"} | |||
ifError={(() => <p>There was an error loading the events list. Please try again later!</p>)} |
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 use this for events on the subject-stage landing pages too?
background-color: white; | ||
box-shadow: 0 2px 30px 0 rgba(0, 0, 0, 0.08); |
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.
Hm. That CSS block should be switched to the form styling, otherwise it doesn't give a background to hints..? Also fixes that issue as then the outline
and border-radius
are applied to the same element. Not sure why it wasn't.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## redesign-2024 #1468 +/- ##
=================================================
- Coverage 40.60% 40.59% -0.01%
=================================================
Files 494 495 +1
Lines 22052 22106 +54
Branches 6525 7325 +800
=================================================
+ Hits 8954 8975 +21
- Misses 13061 13094 +33
Partials 37 37 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
…vements-9 [VRT] Update baselines for redesign/general-improvements-9
Yet more bugfixes!
Of the larger changes:
doc
on page switch so that calls todoc.get
(used in various content components to determine the page type) are not stale;<Form/>
) so that hints are included and appear in the same "block";