Skip to content

[IMPROVE] Add emoji picker to iOS#4366

Merged
reinaldonetof merged 25 commits into
developfrom
improve.ios-keyboard
Aug 8, 2022
Merged

[IMPROVE] Add emoji picker to iOS#4366
reinaldonetof merged 25 commits into
developfrom
improve.ios-keyboard

Conversation

@reinaldonetof

@reinaldonetof reinaldonetof commented Jul 14, 2022

Copy link
Copy Markdown
Contributor

Proposed changes

Use the same keyboard layout for both Andorid and iOS. Also, add the emoji picker on keyboard.

This task is dependent on the merge of the PR RocketChat/react-native-ui-lib#6

Issue(s)

How to test or reproduce

  • Open any room
  • Click on the "icon" button to open the emoji picker

Screenshots

Before

After

Screen Shot 2022-07-14 at 16 08 12

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

return <ActionsButton onPress={showMessageBoxActions} />;
}
return <View style={styles.buttonsWhitespace} />;
return !isIOS ? <View style={styles.buttonsWhitespace} /> : null;

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.

Why do we need this?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I couldn't see a visual difference in this case here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The VIEW was implemented before only for Android, because of this I created this conditional. I was not able to find why this was implemented only for android. But I remember that at some point a long time ago @diegolmello commented with me about this view. So instead create a regression, I implemented this conditional.

Comment thread app/containers/MessageBox/index.tsx Outdated
Comment thread app/containers/MessageBox/index.tsx Outdated
Comment thread app/containers/MessageBox/index.tsx Outdated
Comment thread app/views/RoomView/index.tsx Outdated
@diegolmello diegolmello changed the title [IMPROVE] Add emoji picker to iOS and unify the layouts [IMPROVE] Add emoji picker to iOS Jul 18, 2022
Comment thread app/views/RoomView/RightButtons.tsx
Comment thread app/containers/MessageBox/index.tsx Outdated
Comment thread app/containers/EmojiPicker/EmojiCategory.tsx
Comment thread app/views/RoomView/RightButtons.tsx
return <ActionsButton onPress={showMessageBoxActions} />;
}
return <View style={styles.buttonsWhitespace} />;
return !isIOS ? <View style={styles.buttonsWhitespace} /> : null;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I couldn't see a visual difference in this case here.

@reinaldonetof reinaldonetof merged commit e38aedc into develop Aug 8, 2022
@reinaldonetof reinaldonetof deleted the improve.ios-keyboard branch August 8, 2022 18:38
diegolmello pushed a commit that referenced this pull request Sep 12, 2022
* [IMPROVE] Add emoji picker to iOS

* clean left and right buttons

* fix the redux in emojipicker

* fix behavior when emoji keyboard is openning

* added isIOS

* fix show reactions when emoji is open

* minor tweak

* add provider

* fix baseurl

* minor tweak

* create closeEmojiAndAction and added to record

* fix actionsheet for omnichannel

* fix action sheet

* fix close emoji when navigate to other screen

* added iactionsheetprovider to roomview

* clean variables

* fix theme

* close the emojikeyboard when click on message

* apoint package.json to new pr

* fix branch

* fix package.json
ivnxyz pushed a commit to NextiaDev/Rocket.Chat.ReactNative that referenced this pull request May 26, 2023
* [IMPROVE] Add emoji picker to iOS

* clean left and right buttons

* fix the redux in emojipicker

* fix behavior when emoji keyboard is openning

* added isIOS

* fix show reactions when emoji is open

* minor tweak

* add provider

* fix baseurl

* minor tweak

* create closeEmojiAndAction and added to record

* fix actionsheet for omnichannel

* fix action sheet

* fix close emoji when navigate to other screen

* added iactionsheetprovider to roomview

* clean variables

* fix theme

* close the emojikeyboard when click on message

* apoint package.json to new pr

* fix branch

* fix package.json
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