Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Conversation

@tdurnford
Copy link
Collaborator

Description

Fixed the 'Validation Rules' array styling so the expression expands to use the full width of the property editor and errors render below the text field instead of next to it.

Task Item

Closes #3187

Screenshots

ca0007b2-7a78-4ff7-92e8-2f1f56ebf260

beyackle
beyackle previously approved these changes May 26, 2020
Copy link
Contributor

@beyackle beyackle left a comment

Choose a reason for hiding this comment

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

This change looks good, but at some later point I want to come back to what's going on with types like string | false | undefined on that label prop. I'm sure it works, but it looks weird.

@tdurnford
Copy link
Collaborator Author

@beyackle Yes, I agree - the label prop is not very intuitive and has room to be improved. Here's a quick overview of the prop's current behavior.

  • undefined: If the label is undefined, it uses the default label from the SDK Schema
  • string: Overrides the default label from the SDK Schema
  • false: If the label is false, the label is hidden in the Property Editor. This is typically the case in arrays, so we don't have repeated labels.

@cwhitten cwhitten merged commit d154a44 into microsoft:master May 26, 2020
@tdurnford tdurnford deleted the fix/array-of-expressions branch June 3, 2020 21:08
lei9444 pushed a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
* fixed styling

* fixed styling

* lint fix

Co-authored-by: TJ Durnford <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[PropertyEditor] Prompt validator field styling

3 participants