Skip to content

Add details labels to the docs #125

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

Merged
merged 3 commits into from
Apr 8, 2022
Merged

Add details labels to the docs #125

merged 3 commits into from
Apr 8, 2022

Conversation

felipecruz91
Copy link
Contributor

@felipecruz91 felipecruz91 commented Apr 7, 2022

docker run --rm -it -p 8000:8000 -v ${PWD}:/docs squidfunk/mkdocs-material

Visit: http://0.0.0.0:8000/extensions/labels/

image

@@ -1,12 +1,16 @@
Labels are specified in the extension's `Dockerfile` and are use to provide information about the extension.

| Label | Required | Description | Example |
| Label <div style="width:300px"></div>| Required | Description | Example |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added a fixed width to the label column so the label value can be displayed properly.

| `com.docker.extension.screenshots` | No | A list of images in the shape of comma-separated URLs to be displayed in the details dialog, in the order on which they’ve been defined. | `https://foo.bar/image1.png,https://foo.bar/image2.jpg` |
| `com.docker.extension.detailed-description` | No | Additional information in plain text or HTML about the extension to be displayed in the details dialog. | `My detailed description` or `<h1>My detailed description</h1>` |
| `com.docker.extension.publisher-url` | No | The publisher website URL to be displayed in the details dialog. | `https://foo.bar` |
| `com.docker.extension.additional-urls` | No | A list of additional URLs to be displayed displayed in the details dialog, in the order on which they’ve been defined. | `[{"title":"Twitter profile","url":"https://twitter.com/john"},{"title":"Sponsor my work","url":"https://github.com/sponsor/john"}]` |
Copy link
Contributor

Choose a reason for hiding this comment

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

This is where I would change the examples to propose "Documentation", "Support", "Terms of services" links. And maybe in the text details, mention the 4 typical ones we have thought about

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've updated the copy and the examples, thanks

@felipecruz91 felipecruz91 force-pushed the feature/details-labels branch from 2eaa171 to 74358e2 Compare April 7, 2022 11:20
@felipecruz91 felipecruz91 requested a review from gtardif April 7, 2022 11:20
felipecruz91 and others added 2 commits April 8, 2022 08:55
Co-authored-by: Benjamin Grandfond <benjamin.grandfond@gmail.com>
@felipecruz91 felipecruz91 requested a review from benja-M-1 April 8, 2022 06:58
@felipecruz91 felipecruz91 merged commit b214f2c into main Apr 8, 2022
@felipecruz91 felipecruz91 deleted the feature/details-labels branch April 8, 2022 15:15
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