Skip to content
This repository has been archived by the owner on Apr 1, 2020. It is now read-only.

don't output buttons in head css #438

Merged
merged 1 commit into from
Sep 16, 2016
Merged

Conversation

wheresrhys
Copy link
Contributor

We only use the classes for stream page pagination and a handful of places in myft-page.

In v3 I think we should stop outputting button classes, but for now this hack at least takes them out of the critical path

@i-like-robots

@wheresrhys wheresrhys merged commit 347f2b6 into master Sep 16, 2016
leafrogers added a commit that referenced this pull request Sep 16, 2016
* master: (24 commits)
  `&&` disappears from the terminal when echoed (#206)
  added maybe alias to nUiCritical to remind users that it's configurable (#440)
  Revert "don't output buttons in head css" (#439)
  don't output buttons in head css (#438)
  tweaked css build for more accurate metrics
  tweaked css build for more accurate metrics
  better purge fail logging
  removed simple from critical header styles (#435)
  Remove irrelevant tests
  Fix linting
  Fix homepage site/zone
  force latest o-video
  Remove drawer and sticky header from critical css (#431)
  Sticky-right-rail ad - refactor to work with sticky-banner
  Sticky-right-rail ad - refactor to work with sticky-banner
  Sticky-right-rail ad - refactor to work with sticky-banner
  Sticky-right-rail ad - refactor to work with sticky-banner
  Sticky-right-rail ad - refactor to work with sticky-banner
  Sticky-right-rail ad - refactor to work with sticky-banner
  Sticky-right-rail ad - refactor to work with sticky-banner
  ...
@andygout andygout deleted the rhys/non-critical-buttons branch December 5, 2016 11:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant