Adjust to code review

Remove key prop from Stepper.Item

Adjust to code review:

• change closed to secret
• replace threadTypesPerVisibility with getThreadType function

Change closed to secret

Rebase before landing

Rebase before landing

Thanks! Now it should work

Adjust to code review:

• set error message as optional nullable value

Adjust to code review:

• move Stepper.Items directly to Stepper.Container
• replacing empty string with stringForUser function

Adjust to code review:

• fix typo
• change matrix name to threadTypesPerVisibility
• use createLoadingStatusIndicator instead of using component state

Rebase before landing

Adjust to code review:

• fix stepContainer height
• change <not specified to community
• filter empty-username users from list
• add new css variables

Adjust to code review:

• add margin to gap between buttons and the step content
• remove additional spaces
• simplify code
• use classnames instead of string literals

Adjust to code review:

• set button type as submit
• flex adjustments
• change height: 100% to flex: 1
• use classnames for containerStyle
• round fractional height

Rebase

In D5222#154090, @tomek wrote:

In D5222#153960, @atul wrote:

Based on the video you provided, should we also gray out the Delete button in the Delete tab if the password input field is empty?

Yeah, sounds like a good idea. We should create a new task for it and implement it when appropriate. @jakub could you create a task?

Adjust to code review and rebase

Adjust to code review:

• change SubmitButton name to SubmitSection
• remove min-height property
• simplify code and remove unnecessary spaces

Adjust to code review:

• Change ActionButtonsProps to readonly props
• use new name prop instead of using key
• remove errorMessage fixed size

Adjust to code review:

• change flag icon to megaphone,
• remove unnecessary whitespaces,
• simplify code,
• memoize steps,
• use css variables from typography.css and theme.css,
• adjust userInfos in MembersList to new type of AddMembersList

Rebase before landing

Adjust to code review: simplify code, move prepareButton function outside StepperItem
component, remove unnecessary css prop

Memoize sortedGroupedUsersList

Change MemberGroupItem types and memoize sortedGroupedUsersList

Rebase before landing

Rebase

Move error message to left side

Add div for error messages

Add optional LoadingIndicator to buttons

Rebase

Rebase

Rebase and move additional types declarations before Props type

After discussion in real life, I decided to abandon this diff and include this component directly to ComposeSubchannel

Set readOnly as optional param and use $Keys instead of hardcoding IconPosition type

Rebase before landing

Changing % to px units, set default value of type param, move iconPositionClassNames outside of the component and adjust to codebase

Adjust to codebase

By default, this Radio component is only used in and controlled by EnumSettingsOption component. So basically, we don't need to pass any onChange function to them.

Rebase branch and rename ActiveThreadBehaviour and conditionalOnClick