Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

EditMessageComposer: disable Save button until a change has been made #3410

Merged
merged 3 commits into from
Sep 11, 2019

Conversation

t3chguy
Copy link
Member

@t3chguy t3chguy commented Sep 9, 2019

Fixes element-hq/element-web#10809

Also closes the edit event view if someone tries to send an edit which would not change the event at all, instead of doing nothing with no response to the user that their event would remain unchanged.

10809

Signed-off-by: Michael Telatynski 7t3chguy@gmail.com

@t3chguy t3chguy requested a review from a team September 9, 2019 17:14
Copy link
Contributor

@bwindels bwindels left a comment

Choose a reason for hiding this comment

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

lgtm otherwise

src/components/views/rooms/EditMessageComposer.js Outdated Show resolved Hide resolved
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

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

lgtm - would be good to resolve Bruno's comments

Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
Signed-off-by: Michael Telatynski <7t3chguy@gmail.com>
@t3chguy t3chguy force-pushed the t3chguy/disable_save_button_msg_edit_nochange branch from fa20749 to 792b709 Compare September 11, 2019 16:37
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

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

lgtm, but please avoid force pushing stuff. From a review perspective, it means the whole thing needs to be re-reviewed from scratch.

@t3chguy
Copy link
Member Author

t3chguy commented Sep 11, 2019

Is there a cleaner way to rebase? Pull develop and push made the PR unclean for some reason this time around because it included all the pulled in commits for whatever reason

@turt2live
Copy link
Member

usually merge develop into the feature branch. It creates a squiggly line if using gitkraken, but its fine

eg:
image

@turt2live turt2live merged commit ab6215f into develop Sep 11, 2019
@t3chguy t3chguy deleted the t3chguy/disable_save_button_msg_edit_nochange branch May 25, 2020 18:08
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.

Save button should use a disabled state when the edit UI is first shown
3 participants