Andal.LND/frontend-app-learning
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

add black underline when value prop flyover is open

Browse Source
This commit is contained in:
Emma Green
2021-01-02 22:15:16 -05:00
committed by Emma Green
parent db4b4d18cc
commit 9e0f5d7e22
2 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
src/courseware/course/Course.jsx
View File

@@ -99,6 +99,7 @@ function Course({
sequenceId={sequenceId}
toggleREV1512Flyover={toggleREV1512Flyover} /* This line should be reverted after REV1512 experiment */
REV1512FlyoverEnabled={REV1512FlyoverEnabled} /* This line should be reverted after REV1512 experiment */
isREV1512FlyoverVisible={isREV1512FlyoverVisible} /* This line should be reverted after REV1512 experiment */
/>
<AlertList topic="sequence" />
<Sequence

4
src/courseware/course/CourseBreadcrumbs.jsx
View File

@@ -38,6 +38,7 @@ export default function CourseBreadcrumbs({
sequenceId,
toggleREV1512Flyover, /* This line should be reverted after the REV1512 experiment */
REV1512FlyoverEnabled, /* This line should be reverted after the REV1512 experiment */
isREV1512FlyoverVisible, /* This line should be reverted after the REV1512 experiment */
}) {
const course = useModel('courses', courseId);
const sequence = useModel('sequences', sequenceId);
@@ -90,7 +91,7 @@ export default function CourseBreadcrumbs({
<div
className="toggleFlyoverButton"
aria-hidden="true"
style={{ marginLeft: 'auto', marginTop: '-16px' }}
style={{ marginLeft: 'auto', marginTop: '-16px', borderBottom: isREV1512FlyoverVisible() ? '4px solid #00262b' : 'none' }}
onClick={() => {
toggleREV1512Flyover();
}}
@@ -111,6 +112,7 @@ CourseBreadcrumbs.propTypes = {
sequenceId: PropTypes.string,
toggleREV1512Flyover: PropTypes.func.isRequired, /* This line should be reverted after the REV1512 experiment */
REV1512FlyoverEnabled: PropTypes.bool.isRequired, /* This line should be reverted after the REV1512 experiment */
isREV1512FlyoverVisible: PropTypes.func.isRequired, /* This line should be reverted after the REV1512 experiment */
};
CourseBreadcrumbs.defaultProps = {