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

Overriding global-transition sass variable does not disable page animations #1146

Closed
4 tasks done
rightpad opened this issue Aug 2, 2017 · 1 comment
Closed
4 tasks done

Comments

@rightpad
Copy link
Contributor

rightpad commented Aug 2, 2017

  • This is a question about using the theme.
  • I have updated all gems with bundle update.
  • I have tested locally with bundle exec jekyll build.
  • I believe this to be a bug with the theme --- not Jekyll, GitHub Pages or one of the bundled plugins.

Environment information

  • Minimal Mistakes version: 4.4.2
  • github-pages or jekyll gem version: jekyll 3.5.1
  • Operating system: macOS 10.12.6

Expected behavior

Disabled page animations for the theme.

Steps to reproduce the behavior

  1. Use gem based version of theme
  2. Copy file assets/css/main.scss from gem bundle to local site repo (preserving directory structure)
  3. Make the contents of the file the following:
---
# main.scss
---

@charset "utf-8";

/* Theme overrides */
$global-transition : none;

@import "minimal-mistakes";
  1. Run jekyll serve
  2. Navigate to localhost:4000 and observe page animations
@mmistakes
Copy link
Owner

The $global-transition variable isn't used everywhere, so setting it to none won't disable all animations.

For example the page fade in animation is applied to the #main element and doesn't use the global transition.

#main {
  @include container;
  @include clearfix;
  margin-top: 2em;
  padding-left: 1em;
  padding-right: 1em;
  animation: intro 0.3s both;
  ...
}

You'll need to override all of those as well. Unless someone wants to create a pull request that swaps animation: intro 0.3s both with a new Sass variable that can be set...

$intro-transition: intro 0.3s both;

#main {
  animation: $intro-transition;
  ...
}

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

No branches or pull requests

2 participants