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

Sass & Class Renames #759

Merged
merged 50 commits into from
Feb 19, 2019
Merged

Sass & Class Renames #759

merged 50 commits into from
Feb 19, 2019

Conversation

sagalbot
Copy link
Owner

@sagalbot sagalbot commented Feb 17, 2019

New

  • adds sass-loader
  • adds postcss with:
    • autoprefixer
    • cssnano
  • all styles have been pulled out of the component and organized into scss modules
  • common properties have been extracted into !default scss variables prepended with $vs-
  • add CSS variables
  • spinner slot becomes scoped, receives loading boolean prop

Enhancements

Refined selectors to use a single level of specificity, expect when global component states like vs--searching are involved. The refined selectors shave ~2.5kb off the build, and removes a lot of friction when attempting to override component styles.

Changes

  • selected-tag class renamed to vs__selection
  • all classes have been updated to use the vs__ prefix
  • all CSS selectors have been refactored to use a single selector where ever possible
    ex) .v-select .selected-tag becomes .vs__selection
  • full list of selector changes added to v3 draft release notes
  • remove dropdown and rtl classes from el
  • test default appearance within framemorks:
    • bootstrap 3
    • bootstrap 4
    • Bulma
    • Foundation
  • simplify dropdown ul > li > a structure by removing unnecessary a, refactor all selectors to use .vs__dropdown-option
  • dropdownClasses() renamed to componentState()
    • all state classes are prefixed with vs--
  • remove maxHeight prop, control via CSS
  • use SVG for deselect and open-indicator

Closes #574 #662 #677 #535 #468

Related #596 #451 #447

@sagalbot sagalbot added this to the v3.0 milestone Feb 17, 2019
@sagalbot sagalbot self-assigned this Feb 17, 2019
@sagalbot
Copy link
Owner Author

@stevenharman @eriknygren would really appreciate any thoughts either of you have on this one – it's a big change.

@sagalbot sagalbot added the RFC label Feb 17, 2019
@sagalbot sagalbot marked this pull request as ready for review February 19, 2019 01:28
Repository owner deleted a comment from codecov bot Feb 19, 2019
@codecov
Copy link

codecov bot commented Feb 19, 2019

Codecov Report

❗ No coverage uploaded for pull request base (release/v3.0@06800bf). Click here to learn what that means.
The diff coverage is 100%.

Impacted file tree graph

@@               Coverage Diff               @@
##             release/v3.0     #759   +/-   ##
===============================================
  Coverage                ?   88.29%           
===============================================
  Files                   ?        4           
  Lines                   ?      205           
  Branches                ?       56           
===============================================
  Hits                    ?      181           
  Misses                  ?       12           
  Partials                ?       12
Impacted Files Coverage Δ
src/components/Select.vue 85.97% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 06800bf...145e621. Read the comment docs.

@sagalbot sagalbot merged commit dc91310 into release/v3.0 Feb 19, 2019
@eriknygren
Copy link
Contributor

@eriknygren I was dragging my feet a bit to finish the last 20% of this today, and your review & comments are a huge motivator! Really appreciate the comments and 👍.

Happy to hear it! Thanks for your effort

sagalbot pushed a commit that referenced this pull request Apr 14, 2019
* origin/release/v3.0: (67 commits)
  ensure #222 stays closed (#828)
  add guide for reduce and taggable (#827)
  Remove foo bar (#826)
  WIP: v3 – remove onSearch callback prop (#811)
  doc content updates
  overhaul selecting docs, updated navigation and URL structure
  bump vue-server-renderer (#807)
  WIP: V3 - Remove index prop, add reduce prop (#800)
  V3/update list items slot (#799)
  update vuepress (#797)
  Update sandbox
  Add patch from #787
  update package deps, remove extra dev options
  V3 - Remove `mutable` class properties plus other misc changes (#781)
  refactor deselect method (#768)
  Sass & Class Renames (#759)
  fix coveralls coverage reporter (#766)
  remove: - express - gitbook - json-loader - markdown-loader - normalize.css - prismjs - vue-resource
  use test-utils beta-20
  fix failing test
  ...

# Conflicts:
#	docs/gitbook/README.md
#	docs/homepage/home.html
#	src/components/Select.vue
@sagalbot sagalbot deleted the ft/sass-build branch May 4, 2019 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants