-
Notifications
You must be signed in to change notification settings - Fork 2
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
[do not merge] Switch pages theme #48
base: main
Are you sure you want to change the base?
Conversation
Lets maybe check if everything is correct in the custom theme before we merge :) But else looks good! |
7fdd22c
to
e63e44a
Compare
This reverts commit 03fbc41.
@MahrRah I managed to accomplish what we wanted without using our custom theme. Instead, I changed to use the dark cayman theme we wanted and just added a custom layout to make sure we always show the site title instead of the page title :) |
This reverts commit d9130eb.
This reverts commit c6a4da0.
cursor: pointer; color: $ms-lightblue; h
@MahrRah indeed there is a problem with dropdowns. I tried making this work but couldn't. At this point I've spent so much time on this that I'd say we have other lower hanging fruits and more important tasks than changing the theme :D. Unless you have an idea how to make the dropdown work, then feel free to update of course. |
Changes the theme to dark cayman using the modified theme from https://github.com/observability-lab-cse/lab-theme
Tested the theme and it looks ok; every section still shows the title 'Observability Workshop' as expected