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

#1138 starting classic-ui page templates #1139

Open
wants to merge 13 commits into
base: 6.0
Choose a base branch
from
Open

Conversation

flipmcf
Copy link

@flipmcf flipmcf commented Jan 15, 2022

Thank you for your contribution to the Plone Documentation.

Before submitting this PR, please make sure:

No - I am using markdown for plone 6 docs. Let's update this ^^^

Fixes: #

Improves: #1138 #1137

Changes proposed in this pull request:

*The page-templates directory under classic-ui
*The basic structure
*getting the PR process down.

@netlify
Copy link

netlify bot commented Jan 15, 2022

Deploy Preview for 6-docs-plone-org ready!

Name Link
🔨 Latest commit adb0571
🔍 Latest deploy log https://app.netlify.com/sites/6-docs-plone-org/deploys/662ed040eb018c000806999b
😎 Deploy Preview https://deploy-preview-1139--6-docs-plone-org.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Contributor

@stevepiercy stevepiercy left a comment

Choose a reason for hiding this comment

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

I did a quick review. I put in a couple of suggestions for your consideration.

I can do a full review once it is out of draft status.

.gitignore Outdated Show resolved Hide resolved
docs/classic-ui/page-templates/dtml.md Outdated Show resolved Hide resolved
docs/classic-ui/page-templates/index.md Outdated Show resolved Hide resolved
docs/classic-ui/page-templates/index.md Outdated Show resolved Hide resolved
docs/classic-ui/page-templates/index.md Outdated Show resolved Hide resolved
Co-authored-by: Steve Piercy <web@stevepiercy.com>
@MrTango MrTango self-assigned this Aug 29, 2022
@flipmcf flipmcf marked this pull request as ready for review November 28, 2022 15:58
insertion. To avoid this, use the ``structure:`` prefix:

```xml+genshi
<div>${structure: ...}</div>
Copy link
Member

Choose a reason for hiding this comment

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

This doesn't work as shorthand notation. See zopefoundation/Zope#1077. You have to use <div tal:content="structure ..." /> instead.

@stevepiercy stevepiercy changed the base branch from 6-dev to 6.0 April 28, 2024 21:02
@stevepiercy
Copy link
Contributor

@flipmcf @MrTango what's the status of this PR? Let's finish it or close it as "won't fix". I updated the PR to point to the correct branch 6.0 and resolved conflicts.

Copy link
Contributor

@stevepiercy stevepiercy left a comment

Choose a reason for hiding this comment

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

Update URL and markup to Chameleon docs.

docs/classic-ui/page-templates/index.md Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Status: In Progress
Development

Successfully merging this pull request may close these issues.

None yet

4 participants