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

Render 'Details' section with property list key information. #987

Merged

Conversation

sofiaromorales
Copy link
Contributor

Bug/issue #, if applicable: rdar://131373480

Summary

Add a translator to render the already existing Details section. This section will display the data of the new SymbolKit mixin plistDetails.

The Details section contains:
1.Type - Type of the symbol.
2. Key (optional) - Name of the symbol if the title of the page is overwritten and does not shows the key name.
3. Name (optional) - A human-readable name of the key.

Details section displaying the human-readable name:
Screenshot 2024-07-15 at 15 25 54

Details section displaying the key name:
Screenshot 2024-07-15 at 15 26 07

Dependencies

This change depends on a new commit of SymbolKit.

Testing

Steps:

  1. Preview the attached documentation catalog.
  2. Assert that symbols Foo and Bar show the Details section.

plist.docc.zip

Checklist

Make sure you check off the following items. If they cannot be completed, provide a reason.

  • Added tests
  • Ran the ./bin/test script and it succeeded
  • Updated documentation if necessary

To better support the documentation of property list keys this adds the  a new section (Details) that renders the mixin information for the key 'plistDetails'. This is used to better  describe property list keys in a human-friendly way.

rdar://131373480
…morales/swift-docc into 131373480/render-details-section
Copy link
Contributor

@d-ronnqvist d-ronnqvist left a comment

Choose a reason for hiding this comment

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

This looks good with a questions about if there's more that we should cover in that test.

@sofiaromorales
Copy link
Contributor Author

@swift-ci please test

@sofiaromorales
Copy link
Contributor Author

@swift-ci please test

…morales/swift-docc into 131373480/render-details-section
…morales/swift-docc into 131373480/render-details-section
@sofiaromorales
Copy link
Contributor Author

@swift-ci please test

@sofiaromorales
Copy link
Contributor Author

@swift-ci please test

@sofiaromorales sofiaromorales merged commit 2e3e73a into swiftlang:main Jul 19, 2024
2 checks passed
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.

2 participants