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

Underline support in text runs #3051

Closed
7 tasks done
productboard-ac opened this issue Jun 13, 2019 · 4 comments · Fixed by #4205
Closed
7 tasks done

Underline support in text runs #3051

productboard-ac opened this issue Jun 13, 2019 · 4 comments · Fixed by #4205

Comments

@productboard-ac
Copy link
Collaborator

productboard-ac commented Jun 13, 2019

Release Renderer status Tasks status
1.3 ✔️ .NET (#3215)
✔️ Android (#3216)
✔️ iOS (#3217)
✔️ TS (#3218)
✔️ UWP (#3219)
✔️ Shared (#3220)
✔️ Designer (#3221)

Summary

Add an underline property to TextRun

Requirements

  • Be able to underline a text run within a RichTextBlock

Schema/spec

https://github.com/microsoft/AdaptiveCards/pull/3086/files

Proposed changes

Add an underline boolean property to TextRun. Note that depending on #3071 this property might also be added to TextBlock.

Example

image

{
    "type": "AdaptiveCard",
    "body": [
        {
            "type": "RichTextBlock",
            "inlines": [
                {
                    "type": "TextRun",
                    "text": "This is the first inline. "
                },
                {
                    "type": "TextRun",
                    "text": "We support underlining",
                    "underline": true
                },
                ", ",
                {
                    "type": "TextRun",
                    "color": "Good",
                    "text": "colors, "
                },
                {
                    "type": "TextRun",
                    "text": "both regular and subtle...",
                    "isSubtle": true
                }
            ]
        }
    ],
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.2"
}

Host Config

N/A

Down-level impact

Minimal. The underline won't appear, but text still will appear.

Host burden

N/A

Auto-generated task status

  • Shared
  • Designer
  • .NET
  • Android
  • iOS
  • TS
  • UWP
@productboard-ac
Copy link
Collaborator Author

@andrewleader
Copy link
Contributor

andrewleader commented Jul 12, 2019

7/12/2019 review

  • Will there ever be double underline?
    • Consensus is we can support that in future if need be, by upgrading the property to also be an enum so you could specify double

Proposal and spec approved, although the TextBlock aspect is dependent on the other convergence proposal #3071

@ghost
Copy link

ghost commented Dec 6, 2019

This issue has been automatically marked as stale because it has not had any activity for 5 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants