Skip to content

Commit 9e20b70

Browse files
committed
Fix an invalid line of scss that breaks on upgrade
Original commit title: "hide course menu hamburger in breadcrumb on personal files page” This was invalid sass syntax and will break if we upgrade node-sass Original test plan: - as a user with the new ui - navigate to "account" from the left-had nav - click the "files" link (the page should now hide the course menu hamburger icon in the breadcrumb) note: - the breadcrumb alignment was also corrected in this commit old view: http://screencast.com/t/0pNNuheOl8hP new view: http://screencast.com/t/jmor0IBA - lastly, ensure this change did not affect the hambuerger menu on any other pages Change-Id: I14376de6104b88b14621029516bb233fdd6d47db Reviewed-on: https://gerrit.instructure.com/104015 Tested-by: Jenkins Reviewed-by: Pam Hiett <phiett@instructure.com> Product-Review: Ryan Shaw <ryan@instructure.com> QA-Review: Ryan Shaw <ryan@instructure.com>
1 parent d0c7237 commit 9e20b70

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

app/stylesheets/pages/react_files/_react_files.scss

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ body.files {
2424
}
2525

2626
// hides the course menu hamburger in the personal files breadcrumb
27-
body:not(.with-left-side) & .ic-app-nav-toggle-and-crumbs--files {
27+
body:not(.with-left-side) .ic-app-nav-toggle-and-crumbs--files {
2828
> .ic-app-course-nav-toggle {
2929
display: none;
3030
}

0 commit comments

Comments
 (0)