Skip to content
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

justify-content for navbar-left in bs-compat.scss #4054

Open
wants to merge 1 commit into
base: release-10.1
Choose a base branch
from

Conversation

ThoWagen
Copy link
Contributor

@ThoWagen ThoWagen commented Nov 4, 2024

Analogous to navbar-right this PR adds justify-content to the navbar-left class in bs3-compat.scss.

@demiankatz
Copy link
Member

@ThoWagen, should we rebase this onto release-10.1 for inclusion in the 10.0.1 release, or is it okay to wait until release 11.0? Since the bootstrap3 theme will be removed in the 11.0 release, perhaps it is worth backporting... but maybe it's useful for backward compatibility in 11.0 regardless of the status of the bootstrap3 theme.

@ThoWagen ThoWagen changed the base branch from dev to release-10.1 November 4, 2024 14:47
@ThoWagen
Copy link
Contributor Author

ThoWagen commented Nov 4, 2024

@demiankatz That makes sense! I rebased it.

@ThoWagen ThoWagen changed the base branch from release-10.1 to dev November 4, 2024 15:23
@ThoWagen
Copy link
Contributor Author

ThoWagen commented Nov 4, 2024

Sorry, I misunderstood how the rebasing works. Reverted that for now.

@demiankatz
Copy link
Member

@ThoWagen, it should just be a matter of resetting to the release-10.1 branch, reapplying your SCSS change, and then running the build script. If you want me to do that, I'm happy to help!

@ThoWagen ThoWagen force-pushed the pull-request/justify-content-navbar-left branch from 60f48e0 to 18a0570 Compare November 5, 2024 08:32
@ThoWagen ThoWagen changed the base branch from dev to release-10.1 November 5, 2024 08:32
@ThoWagen
Copy link
Contributor Author

ThoWagen commented Nov 5, 2024

@demiankatz Thanks! The rebasing is done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants