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

Fix facet price filter label spacing #300

Merged
merged 4 commits into from
Aug 3, 2021
Merged

Conversation

tauthomas01
Copy link
Contributor

Why are these changes introduced?

Fixes #241

The goal of this PR is to fix the spacing on the "facet price" label on the collection filters.

Before

alt

After

image

What approach did you take?

  • Adjust CSS spacing

Demo links

Checklist

tyleralsbury
tyleralsbury previously approved these changes Aug 3, 2021
Copy link
Contributor

@tyleralsbury tyleralsbury left a comment

Choose a reason for hiding this comment

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

Tested, looks good.

@LucasLacerdaUX LucasLacerdaUX self-requested a review August 3, 2021 14:57
@LucasLacerdaUX LucasLacerdaUX self-assigned this Aug 3, 2021
Copy link
Contributor

@LucasLacerdaUX LucasLacerdaUX left a comment

Choose a reason for hiding this comment

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

One minor change :D

Good job on fixing the spacing. Looks much better now.

@tauthomas01
Copy link
Contributor Author

Thanks Lucas, I did not know that filter priority was happening in Liquid.

alt

Copy link
Contributor

@LucasLacerdaUX LucasLacerdaUX left a comment

Choose a reason for hiding this comment

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

Looks good now! :DD

@tauthomas01 tauthomas01 merged commit 7c130e8 into main Aug 3, 2021
@tauthomas01 tauthomas01 deleted the fix-price-facet-spacing branch August 3, 2021 20:18
maplerock added a commit to fellowsmedia/nsra_shopify_2021 that referenced this pull request Aug 12, 2021
* main: (23 commits)
  Cart content update (Shopify#370)
  Footer spacing and alignment adjustments (Shopify#369)
  Product Template UI polish updates (Shopify#219)
  footer ui updates (Shopify#318)
  Fix cart improvements empty state (Shopify#319)
  [Announcement] Adjust block id for displaying dynamic names (Shopify#327)
  Update translations: buyer (Shopify#329)
  Revert editor setting changes (Shopify#328)
  Update translations (Shopify#294)
  Fix collection filtering UX (Shopify#268)
  Added page width setting and fixed image quality (Shopify#292)
  Add top border on cart notification when "show separator line" setting is deactivated (Shopify#306)
  movebadge code into base.css (Shopify#313)
  Collage UI bug fixes (Shopify#308)
  Customer account UI polish (Shopify#177)
  Added custom liquid block to product page (Shopify#269)
  Fix disclosure icon (Shopify#310)
  Fix facet price filter label spacing (Shopify#300)
  Fix duplicate search icon when header logo is set to "Top center" (Shopify#252)
  Add card outline setting (Shopify#239)
  ...
phapsidesGT pushed a commit to Gravytrain-UK/gt-shopify-dawn-theme that referenced this pull request Sep 3, 2024
* Fix facet price filter label spacing

* Move currency label out of the wrapperto fix the absolute position

* add strip_html to the message when filtering price

* apply strip_html before | escape filter
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.

Filtering: Price Input Fields - Adjust text spacing
3 participants