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

Change example of bard set partials to new antlers format #8

Closed
twd3 opened this issue Oct 25, 2019 · 4 comments
Closed

Change example of bard set partials to new antlers format #8

twd3 opened this issue Oct 25, 2019 · 4 comments

Comments

@twd3
Copy link
Contributor

twd3 commented Oct 25, 2019

On this page: https://statamic.dev/fieldtypes/bard#templating

You should change the example that reads

partials/
|-- sets/
|   |-- gallery.html
|   |-- image.html
|   |-- poll.html
|   |-- text.html
|   |-- video.html

To:

resources/views/partials/
|-- sets/
|   |-- gallery.antlers.html
|   |-- image.antlers.html
|   |-- poll.antlers.html
|   |-- text.antlers.html
|   |-- video.antlers.html

This way people know where to put these partials and they'll be reminded of the new antlers naming for them.

@twd3
Copy link
Contributor Author

twd3 commented Oct 25, 2019

Side note: Tip Tap link wrong right under the example above. Use: https://tiptap.scrumpy.io/ You have .com The next Tip Tap link for extensions is good.

@jackmcdade
Copy link
Member

Thanks for the feedback. The bard docs haven’t been written yet though so make sure to check the writing stage on any given page before worrying too much about mistakes :)

@twd3
Copy link
Contributor Author

twd3 commented Oct 25, 2019

Got it.

@jonassiewertsen
Copy link
Contributor

This PR updates the tiptap links:
#240

jonassiewertsen added a commit to jonassiewertsen/docs that referenced this issue Sep 17, 2020
The updated bard syntax is a little shorter and does look cleaner. 

To make it 100% clear where the partials can be stored, I did add the explicit directory. 

This PR closes statamic#8
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 a pull request may close this issue.

3 participants