Skip to content

[IMPROVE] Redesign flow of create room#4381

Merged
diegolmello merged 75 commits into
developfrom
new.flow-create-room
Aug 26, 2022
Merged

[IMPROVE] Redesign flow of create room#4381
diegolmello merged 75 commits into
developfrom
new.flow-create-room

Conversation

@reinaldonetof

@reinaldonetof reinaldonetof commented Jul 21, 2022

Copy link
Copy Markdown
Contributor

Proposed changes

We're proposing lots of UX improvements on this task.
We're going to make use of the new Searchbox, the new forms pattern and improve the UX overall.

Screen Shot 2022-07-21 at 15 14 42

Issue(s)

How to test or reproduce

Screenshots

Before

Screen.Recording.2022-07-21.at.15.11.39.mov

After

Screen.Recording.2022-07-21.at.15.01.01.mov

Types of changes

  • Bugfix (non-breaking change which fixes an issue)
  • Improvement (non-breaking change which improves a current function)
  • New feature (non-breaking change which adds functionality)
  • Documentation update (if none of the other choices apply)

Checklist

  • I have read the CONTRIBUTING doc
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • I have added necessary documentation (if applicable)
  • Any dependent changes have been merged and published in downstream modules

Further comments

Comment thread app/containers/ChipsUserSelected.tsx Outdated
Comment thread app/containers/ChipsUserSelected.tsx Outdated
Comment thread app/containers/ChipsUserSelected.tsx Outdated
Comment thread app/containers/ChipsUserSelected.tsx Outdated
Comment thread app/containers/ChipsUserSelected.tsx Outdated
Comment thread app/i18n/locales/pt-BR.json Outdated
Comment thread app/views/CreateChannelView.tsx Outdated
Comment thread app/views/CreateChannelView.tsx Outdated
Comment thread app/views/CreateChannelView.tsx Outdated
Comment thread app/views/NewMessageView.tsx Outdated
Comment thread app/containers/Chip/Chip.stories.tsx Outdated
Comment thread app/views/CreateChannelView/RoomSettings/SwitchItem.stories.tsx Outdated
Comment thread app/views/CreateChannelView/index.tsx
Comment thread app/views/CreateChannelView/index.tsx Outdated
},
invitedHeader: {
marginTop: 18,
marginHorizontal: 15,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

16 on Figma
Everything that's not a multiple of 4 needs to be reviewed.
Ask me before commiting, if you're in doubt

Comment thread app/views/CreateChannelView/index.tsx Outdated
<Chip text={name} avatar={username} onPress={() => removeUser(item)} testID={`create-channel-view-item-${item.name}`} />
);
},
shallowEqual

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🤔

Comment thread app/views/SelectedUsersView/Header.tsx Outdated
Comment thread app/views/SelectedUsersView/index.tsx Outdated
Comment thread app/views/SelectedUsersView/Header.tsx Outdated
Comment thread app/views/SelectedUsersView/index.tsx Outdated
@diegolmello diegolmello merged commit 9cbffff into develop Aug 26, 2022
@diegolmello diegolmello deleted the new.flow-create-room branch August 26, 2022 19:16
diegolmello added a commit that referenced this pull request Sep 12, 2022
Co-authored-by: Diego Mello <diegolmello@gmail.com>
ivnxyz pushed a commit to NextiaDev/Rocket.Chat.ReactNative that referenced this pull request May 26, 2023
Co-authored-by: Diego Mello <diegolmello@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants