-
Notifications
You must be signed in to change notification settings - Fork 8
Decoupled flyout region #59
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
Open
darko-hrgovic
wants to merge
44
commits into
ubc-web-services:develop
Choose a base branch
from
darko-hrgovic:flyout
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
44 commits
Select commit
Hold shift + click to select a range
718f00f
Update drawer help text to indicate that including a menu block with …
darkodevubc 04557f8
Moved sticky option down into the clf_navigation_option section
darkodevubc 8d3b545
Removed duplicate clf_navoption and clf_secondarynavoption defined la…
darkodevubc adc3faa
Targetting btn-navbar:first-child with right margin so it does not mo…
darkodevubc 20eeb4d
Making the secondary sticky on mobile
darkodevubc ed4695e
Adjusting position of secondary drop-down nav on mobile
darkodevubc 56ed870
Moved secondary so it is also sticky
darkodevubc e011f9a
Remove whitespace in theme-settings.php
darkodevubc a5a1046
Increase z-index of sticky nav
darkodevubc 22b00dc
Adjusted margin of secondary on mobile when it was already done - rev…
darkodevubc af67041
Adding navigation.sticky.less and adjusting z-index of sticky nav
darkodevubc 04f2ea5
Adjusted z-index of sticky nav after adding navigation.sticky.less
darkodevubc 194163c
Add flyout region to .info
darkodevubc 62f96ca
Add flyout option to theme settings
darkodevubc 637776a
Add flyout js and css in template.php
darkodevubc 0a11c80
Add flyout region to page.tpl.php
darkodevubc fad3f89
Add off.canvas.flyout.js
darkodevubc a20d368
Add off.canvas.flyout.less
darkodevubc a6f5ad7
Add off.canvas.flyout.css
darkodevubc 4255816
Fixed conflict with both drawer and flyout using $container_attribute…
darkodevubc 07675fe
Fixed issue with primary nav not hiding when used in drawer and issue…
darkodevubc 8020a23
A few fixes for secondary nav behaviour when drawer and flyout are en…
darkodevubc e0f89fb
Reverted conversion of drawer CSS to LESS because some stuff broke
darkodevubc d95f1e7
Providing dark blue flyout bg boilerplate
darkodevubc 8391e45
Moved flyout button oiut of nav structure so it can be z-indexed belo…
darkodevubc fcdfafa
Added bg to main menu items that do not expand so that flyout menu do…
darkodevubc c59c018
Flyout button z-index only needs to be 1
darkodevubc d63cb66
Cleaned CSS files
darkodevubc 9729f48
Adding .DS_Store to gitignore
darkodevubc 63d5d0d
Fixes for drawer and made the mobile drop-down nav scrollable like th…
darkodevubc e62e38b
flyout changes for develop merge
darkodevubc 5e5c5c1
ubc7-unit-menu transparent background only on mobile where its height…
darkodevubc 028d634
#ubc7-unit-menu should not have a bg on it because it hides everythin…
darkodevubc 155045e
Removed unconditional height 100% on mobile drop-down menu because it…
darkodevubc 1bbff81
Fixed merge conflict and uncommented $fluidcontainerstart because gra…
darkodevubc 0532616
Error finding includes/template-ubc-clf-elements.inc so we add theme …
darkodevubc 657fc4a
Updated help text for clf_use_primary_menu_in_drawer
darkodevubc d9701d0
Changed to drupal_get_path from path_to_theme
darkodevubc 4835546
Removed git conflict
darkodevubc 9b7c2d5
Removed toggle behaviour in flyout js that was killed by webform cond…
darkodevubc c4ff0fd
Merge branch 'flyout' of github.com:darko-hrgovic/Megatron into flyout
darkodevubc 50034db
Cleaned up whitespace and formatting
darkodevubc 88feb8e
Checking stickynav > 0 because modal close button adds numerous zero …
darkodevubc bb2875a
Whitespace and formatting
darkodevubc File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,3 +4,5 @@ node_modules/ | |
|
||
# Postcss-fied CSS files | ||
postcss/ | ||
|
||
.DS_Store | ||
Large diffs are not rendered by default.
Oops, something went wrong.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I'd suggest moving some of the clean-up changes to their own issue so they can be reviewed as one thing.
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.
Understood.
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.
I've pushed the changes wrt to your comments, Joel. Thanks very much for reviewing.