Skip to content

Add the Formattable section to the spec#198

Closed
eemeli wants to merge 1 commit intounicode-org:mainfrom
eemeli:add-formattable
Closed

Add the Formattable section to the spec#198
eemeli wants to merge 1 commit intounicode-org:mainfrom
eemeli:add-formattable

Conversation

@eemeli
Copy link
Copy Markdown
Collaborator

@eemeli eemeli commented Sep 23, 2021

As the proposal's text describes, a Formattable is a wrapper around values and their formatting options. It may be thought of as an abstraction that allows us to define the rest of the specification in a manner that's independent of the actual type of any runtime value.

The text briefly refers to MessageFormatParts. The section defining them has not yet been submitted for review, but some relevant discussion may be found in issue #41. Similarly, while PatternElements are also referred to in the preceding PR #195, they are not yet in the spec. Relevant discussion may be found in #190, which also includes some consideration of Formattables.

In the third proposal by @stasm, a corresponding abstraction uses the name RuntimeValue.

@mihnita
Copy link
Copy Markdown
Collaborator

mihnita commented Sep 29, 2021

I think we are talking about this in #190 ?
So maybe let's not "fork" the discussion and confuse things?

@eemeli
Copy link
Copy Markdown
Collaborator Author

eemeli commented Sep 29, 2021

@mihnita While the discussion in #190 does also cover Formattables, that should not preclude a review of this particular PR.

Copy link
Copy Markdown
Collaborator

@mihnita mihnita left a comment

Choose a reason for hiding this comment

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

@mihnita While the discussion in #190 does also cover Formattables, that should not preclude a review of this particular PR.

It does, because it is not clear at all that this concept is even needed.
And even less what it looks like.

@eemeli
Copy link
Copy Markdown
Collaborator Author

eemeli commented Oct 24, 2022

Closing as out of date after #307.

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