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

Sage 9 - fix bootstrap4 styles #1642

Merged
merged 4 commits into from
May 3, 2016
Merged

Sage 9 - fix bootstrap4 styles #1642

merged 4 commits into from
May 3, 2016

Conversation

PTiCA1
Copy link
Contributor

@PTiCA1 PTiCA1 commented May 2, 2016

No description provided.

@retlehs
Copy link
Member

retlehs commented May 2, 2016

what is this fixing?

@aaemnnosttv
Copy link
Contributor

Class changes in the v4 alpha which break @extends for a few styles due to the parent class no longer existing.

  • .btn-default is now .btn-secondary
  • .thumbnail is now .img-thumbnail (I think?) but @PTiCA1 has something slightly different so I'm not sure if it's the same.

@PTiCA1
Copy link
Contributor Author

PTiCA1 commented May 2, 2016

@aaemnnosttv Yes exactly.
The new version of the bootstrap class names are slightly different.

I just class .thumbnail replacing figures + alternatives .img-thumbnail
http://v4-alpha.getbootstrap.com/content/figures/

In the current version of sage 9 can not perform "run NPM build" with error @extend ...

@extend .img-fluid;

// alternative styles
// @extend .img-thumbnail;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you remove these two comments?

@retlehs
Copy link
Member

retlehs commented May 2, 2016

gotcha, thanks. remove those commented out lines and then this is good to merge

@PTiCA1
Copy link
Contributor Author

PTiCA1 commented May 2, 2016

removed...

@retlehs retlehs merged commit 83ee29f into roots:sage-9 May 3, 2016
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