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

added learn more link to custom liquid #2772

Merged
merged 2 commits into from
Jul 12, 2023

Conversation

lougoncharenko
Copy link
Contributor

PR Summary:

Custom Liquid section info contains a learn more link.

Why are these changes introduced?

Fixes #2726 .

What approach did you take?

I added a learn more tag that links to an informational page about liquid

Visual impact on existing themes

N/A

Testing steps/scenarios

  • Add a custom liquid section
  • Click on the link in the info paragraph

Demo links

Checklist

@lougoncharenko lougoncharenko self-assigned this Jul 5, 2023
@lougoncharenko lougoncharenko linked an issue Jul 5, 2023 that may be closed by this pull request
Copy link
Contributor

@ludoboludo ludoboludo left a comment

Choose a reason for hiding this comment

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

Looking good 👍 There is something that's part of that type of setting which links to the same docs already.

Though I don't know if it's an issue 🤔

@katycobb
Copy link
Contributor

katycobb commented Jul 5, 2023

Looking good 👍 There is something that's part of that type of setting which links to the same docs already.

Though I don't know if it's an issue 🤔

Oh good callout, I do think that's an issue. Can we remove that offsite icon?

@@ -658,7 +658,7 @@
"settings": {
"custom_liquid": {
"label": "Liquid code",
"info": "Add app snippets or other code to create advanced customizations."
"info": "Add app snippets or other code to create advanced customizations. [Learn more](https://shopify.dev/docs/api/liquid)"
Copy link
Contributor

@ludoboludo ludoboludo Jul 5, 2023

Choose a reason for hiding this comment

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

The icon isn't defined at the theme level 🤔 So we would need to work with the folks that do editor work. Maybe @kmeleta would know 😅
cc: @katycobb

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The offsite icon was introduced with the online-store-editor and can't be changed in dawn. We would have to create a separate issue/ticket for this @kimberlyoleiro

@lougoncharenko lougoncharenko merged commit 0df46de into main Jul 12, 2023
@lougoncharenko lougoncharenko deleted the update-custom-liquid-section-with-link branch July 12, 2023 17:06
phapsidesGT pushed a commit to Gravytrain-UK/gt-shopify-dawn-theme that referenced this pull request Sep 3, 2024
* added learn more link to custom liquid

* Update 20 translation files

---------

Co-authored-by: translation-platform[bot] <34770790+translation-platform[bot]@users.noreply.github.com>
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.

Custom Liquid section copy updates
4 participants