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

Update docs_style_guide.md #1378

Merged
merged 4 commits into from
Dec 6, 2019
Merged

Update docs_style_guide.md #1378

merged 4 commits into from
Dec 6, 2019

Conversation

manez
Copy link
Member

@manez manez commented Dec 3, 2019

GitHub Issue: #1040

  • Other Relevant Links (Google Groups discussion, related pull requests, Release pull requests, etc.)

What does this Pull Request do?

Fixes the tip about how to make Tips so that it should actually render properly in the docs Right now, it's not.

What's new?

I think it needs a title to work properly.

How should this be tested?

run mkdocs on the docs and see if the tip presents as a nice blue box, as seen here

Interested parties

@Islandora/8-x-committers

@@ -12,7 +12,7 @@
-- For file name, use underscores between words and prefix all file names with the page name, e.g. context_display_hints.jpg for the image showing how to set display hints in the context menu.
- Use `!!! note` at the start of a paragraph to have it rendered as a note like this:
Copy link
Member

Choose a reason for hiding this comment

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

Maybe this should specify that you need to have a title, as it appears required. Also might be good to explictly display the note below.

Use !!! "Note Title" at the start of a paragraph to have it rendered as a note like this:

!!! "Helpful Tip" I am a helpful tip!

Copy link
Member Author

Choose a reason for hiding this comment

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

Good idea. Done!

Copy link
Member

@whikloj whikloj left a comment

Choose a reason for hiding this comment

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

👍

@whikloj
Copy link
Member

whikloj commented Dec 4, 2019

@manez your branch is out-of-date, could you update so we can merge. It is my arch-nemesis.

@manez
Copy link
Member Author

manez commented Dec 6, 2019

@whikloj all up to date, I think?

@whikloj whikloj merged commit 00719f0 into Islandora:master Dec 6, 2019
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.

3 participants