-
Notifications
You must be signed in to change notification settings - Fork 40
Apply modern table presentation to some more annotations #2957
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
Closed
henrikt-ma
wants to merge
20
commits into
modelica:master
from
henrikt-ma:cleanup/style-missingInnerMessage
Closed
Changes from all commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
11a390a
Clean up formatting of annotations in listings
henrikt-ma 6354d85
Align Library annotation syntax with actual use in the MSL
henrikt-ma ffc8c02
Improve formatting related to missingInnerMessage
henrikt-ma 72e7a19
Remove hyphenated use of 'annotation' according to style guide
henrikt-ma 8d3aeb6
Break too long lines in listings
henrikt-ma 6c62d4b
Deal with bad line breaks by modernizing presentation
henrikt-ma e9490b0
Modernize presentation of 'choces' and 'choicesAllMatching'
henrikt-ma 1859e7f
Modernize presentation of 'singleInstance'
henrikt-ma 4f4f192
Modernize presentation of version handling
henrikt-ma 2351199
Merge remote-tracking branch 'central/master' into cleanup/style-miss…
henrikt-ma 47ed90c
Merge remote-tracking branch 'central/master' into cleanup/style-miss…
henrikt-ma af16ab5
Merge remote-tracking branch 'central/master' into cleanup/style-miss…
henrikt-ma 8c4f75b
Change 'class-modification' -> 'modification' in 'choices-argument'
henrikt-ma a0b1ca0
Clean up white-space inside unassignedMessage string in listing
henrikt-ma 87b8f4a
Define lexical unit PACKAGE-VERSION for contents of package version s…
henrikt-ma dbebe1b
Spell out meaning of \textvisiblespace
henrikt-ma 6033119
Merge remote-tracking branch 'central/master' into cleanup/style-miss…
henrikt-ma c6f4807
Express PRE-RELEASE-VERSION using MAIN-VERSION
henrikt-ma 493441c
Merge remote-tracking branch 'central/master' into cleanup/style-miss…
henrikt-ma fd41229
Use non-breaking space instead of \textvisiblespace
henrikt-ma File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would prefer if we just write that it should be a string or an array of strings, and don't use grammar fragments unless needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are you sure this isn't a good place to use it? It is defined once, used in the definitions of 5 different annotations so far, and also mentioned in the running text. It would take considerably more text and hand waving to do it without giving a name to the idea.
While we might be able to change many of the annotations to the pseudo-code style of presentation, the ones we have here belong to the category where specification using grammar actually makes sense as we can't express the type of the annotation using a Modelica type.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couldn't we just keep the grammar fragment for now, and revisit this once we take the bigger grip on presentation of annotations as decided at the last phone meeting (can't find in what issue that decision was noted)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any chance we can return to this once we turn to the question of how to present the things that can't be expressed easily in the record style established so far?