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

[Android] Support Action.isEnabled #5776

Merged
merged 10 commits into from
May 21, 2021
Merged

[Android] Support Action.isEnabled #5776

merged 10 commits into from
May 21, 2021

Conversation

golddove
Copy link
Member

@golddove golddove commented May 5, 2021

Related Issue

Fixes #5719

Description

  • Re-SWIG'd shared model changes.
  • Elements with selectAction or inlineAction with isEnabled: false are not clickable.
  • RichTextBlock's Inline with selectAction with isEnabled: false are rendered as if there is no selectAction provided
  • Top-level actions and ActionSet actions with isEnabled: false` are not clickable AND get platform's disabled button appearance.
  • Custom actions are expected to determine how to support isEnabled (the SDK doesn't impose any enabling/disabling)

Sample Card

Updated v1.5\Elements\Action.isEnabled sample to include RichTextBlock

How Verified

5719

Microsoft Reviewers: Open in CodeFlow

@golddove golddove requested review from jwoo-msft and RebeccaAnne May 5, 2021 17:06
Copy link
Member

@jwoo-msft jwoo-msft left a comment

Choose a reason for hiding this comment

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

looks great; except for the one question, I had in the review.

@golddove
Copy link
Member Author

golddove commented May 6, 2021

have you gotten VSCode set up with the editorconfig extension?

Yes! Yet, it thinks this file should have tabs (bottom info bar).

also, something something emacs

I'd sooner use vim :)

@paulcam206
Copy link
Member

have you gotten VSCode set up with the editorconfig extension?

Yes! Yet, it thinks this file should have tabs (bottom info bar).

Actually,

[**.{json,html}]
indent_style = tab

also, something something emacs

I'd sooner use vim :)

you can do that in emacs!

Copy link
Member

@jwoo-msft jwoo-msft left a comment

Choose a reason for hiding this comment

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

looks great; except for the one question, I had in the review.

@ghost ghost added the no-recent-activity label May 12, 2021
@ghost
Copy link

ghost commented May 12, 2021

Hi @golddove. This non-spec pull request has had no recent activity for the past 5 days . Please take the necessary actions (review, address feedback or commit if reviewed already) to move this along.

@ghost ghost removed the no-recent-activity label May 13, 2021
@ghost
Copy link

ghost commented May 13, 2021

Hi @golddove; Thanks for taking action on your previously stale pull request. Resetting staleness.

@golddove golddove changed the base branch from main to 1.0.0 May 13, 2021 16:56
@golddove golddove changed the base branch from 1.0.0 to main May 13, 2021 16:56
@golddove golddove requested review from paulcam206 and jwoo-msft May 13, 2021 17:06
@golddove golddove merged commit 083fb72 into main May 21, 2021
@golddove golddove deleted the golddove/5719 branch May 21, 2021 06:06
michaelfarnsworth pushed a commit to michaelfarnsworth/AdaptiveCards that referenced this pull request Nov 10, 2022
* [Android] Support Action.isEnabled

* VSCode will figure out indentation one day...

* Missing SWIG file

* VSCode was right all along

* Outcome of design discussion
rankush pushed a commit to rankush/AdaptiveCards that referenced this pull request May 8, 2024
* [Android] Support Action.isEnabled

* VSCode will figure out indentation one day...

* Missing SWIG file

* VSCode was right all along

* Outcome of design discussion
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.

[Android] Action isEnabled
3 participants