Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TW-1644: allow access to the gallery is not center #1815

Merged
merged 3 commits into from
Jun 17, 2024

Conversation

sherlockvn
Copy link
Contributor

@sherlockvn sherlockvn commented May 29, 2024

MERGE THIS FIRST:

linagora/linagora-design-flutter#44

Related ticket:

#1644

Android:

Screen.Recording.2024-05-29.at.11.21.11.mov

IOS:

Screen.Recording.2024-05-29.at.11.21.11.mov

Copy link

This PR has been deployed to https://linagora.github.io/twake-on-matrix/1815

@sherlockvn sherlockvn force-pushed the TW-1644/allow-access-your-gallery-isnot-center branch from 6d5b67c to a5467a7 Compare May 29, 2024 04:52
@Te-Z
Copy link
Contributor

Te-Z commented May 30, 2024

Quick win to improve UX: IMO the bottom sheet should close when clicking on the "go to settings" button. This way we suggest to the user that he must retry to send something from his gallery

@sherlockvn
Copy link
Contributor Author

Quick win to improve UX: IMO the bottom sheet should close when clicking on the "go to settings" button. This way we suggest to the user that he must retry to send something from his gallery

if user change any permission, the app will be restart and the state of app will lost. so IMO, you don't have to worry about this case

@Te-Z
Copy link
Contributor

Te-Z commented Jun 3, 2024

Quick win to improve UX: IMO the bottom sheet should close when clicking on the "go to settings" button. This way we suggest to the user that he must retry to send something from his gallery

if user change any permission, the app will be restart and the state of app will lost. so IMO, you don't have to worry about this case

Not every time. Look at your demo video

@sherlockvn
Copy link
Contributor Author

sherlockvn commented Jun 4, 2024

Quick win to improve UX: IMO the bottom sheet should close when clicking on the "go to settings" button. This way we suggest to the user that he must retry to send something from his gallery

if user change any permission, the app will be restart and the state of app will lost. so IMO, you don't have to worry about this case

Not every time. Look at your demo video

Okay i fixed it

Screen.Recording.2024-06-04.at.10.25.24.mov
RPReplay_Final1717473141.MP4

@Te-Z
Copy link
Contributor

Te-Z commented Jun 6, 2024

Hi @sherlockvn did you push your changes ?

@sherlockvn sherlockvn force-pushed the TW-1644/allow-access-your-gallery-isnot-center branch from 3bc65c9 to dd399e8 Compare June 7, 2024 03:20
@sherlockvn
Copy link
Contributor Author

Hi @sherlockvn did you push your changes ?

sr i pushed again now

@sherlockvn sherlockvn force-pushed the TW-1644/allow-access-your-gallery-isnot-center branch from dd399e8 to 1876f1a Compare June 11, 2024 02:25
Copy link
Member

@nqhhdev nqhhdev left a comment

Choose a reason for hiding this comment

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

LGTM

@Te-Z Te-Z merged commit e5bb459 into main Jun 17, 2024
7 checks passed
@Te-Z Te-Z deleted the TW-1644/allow-access-your-gallery-isnot-center branch June 17, 2024 14:51
Te-Z pushed a commit that referenced this pull request Jun 18, 2024
* TW-1644: allow access to the gallery is not center

* TW-1644: close the bottomsheet when go to settings

* TW-1644: update style for image picker
Te-Z pushed a commit that referenced this pull request Jun 18, 2024
* TW-1644: allow access to the gallery is not center

* TW-1644: close the bottomsheet when go to settings

* TW-1644: update style for image picker
Te-Z added a commit that referenced this pull request Jun 21, 2024
* TW-1772: Prevent infinity contact loop

* TW-1772: Show recent contact in contact tab

* TW-1772: Showing no contact in new chat screen

* TW-1772: Showing no contact in new group chat screen

* TW-1773: review load 0 more participants

* TW-1773: listen for roomMember change in chat details

* TW-1735: search external contact widget added on search view

* TW-1735: update matrix dart sdk

* TW-1780: Open screen can connect with Twake Workplace when add another account

* TW-1780: Add `app=chat` to parameter whenever call to signup in TWP

* TW-1785: Improve logout for multiple homeserver (#1788)

* Bump version to v2.5.4

* Update privacy welcome screen

* TW-1526: Fix Cannot unmute more than 2 chats at the same time

* Bump version to v2.5.5

* TW-1781: change the ci/cd to use flutter 3.22

* TW-1781: remove warning based on new lint version

* TW-1781: remove deprecated change to have new change

* TW-1781: fix the ci issue

* TW-1781: add deprecated and TODO for background and onBackground color

* TW-1781 Fix ci failed

* Bump version to v2.5.8

* TW-1645: Add counting selected image at send icon (#1803)

* TW-1645: Add counting selected image at send icon

* TW-1645: Add style file for `media_picker_mixin`

* TW-1645: Fix size for counter icon (#1816)

* TW-1615: Support auto jump to lastest message when user send an attachments  (#1810)

* TW-1615: Support auto jump to lastest message when user send an attachment

* TW-1615: Support auto jump to lastest message when use copy/paste

* hot-fix: members list always have to reload (#1801)

* TW-1793: recent chats are not recent (#1800)

* hot-fix: fix the index.html loading failed

* TW-1356: change action for profile tap

* TW-1356: update UI for multiple accounts in setting profile

* TW-1356: fix multiple accout with same homeserver can login

* TW-1356: fix the avatar not updated when app terminated

* hot-fix: fix the problem with index.html file

* TW-1635: remove the messages contains reply from search result (#1819)

* Switched runners to macos-latest (#1820)

* Update metadata to config store kit (#1843)

* TW-1698: Add minWidth for group details (#1809)

* hot-fix: fix the error l10n not found after build runner build (#1855)

* TW-1841: Fix back screen when use go_router and navigator (#1850)

* Removed xcbeautify installation because it's preinstalled on runners (#1865)

* TW-1787: Improve when leave chat (#1842)

* TW-1787: Move UI of action items into context menu widget

* TW-1787: Update context menu for chat screen

* TW-1787: Update context menu for chat list

* TW-1787: Update context menu for pinned screen

* TW-1812 Remove X from the search bar when it's empty (#1852)

* TW-1812 Remove X from the search bar when it's empty

* fixup! TW-1812 Remove X from the search bar when it's empty

* README.md

Updated picture as management asked to do

* Update Feature Graphic for fastlane metadata

* Improve tom bootstrap dialog (#979)

* Create init client dialog on mobile

* Implement new bootstrap dialog for mobile

* Implement new on auth screen for web

* Prevent back button and back android during loading in homeserver picker

* Fix dialog overlap when resizing screen

* Fix dialog overlap when resizing screen

* Handle error for `_getRecoveryWords`

* Handle error for Twake welcome screen

* TW-1644: allow access to the gallery is not center (#1815)

* TW-1644: allow access to the gallery is not center

* TW-1644: close the bottomsheet when go to settings

* TW-1644: update style for image picker

* TW-1602: when typing shift ctrl, the cursor is dissappear (#1853)

* TW-1825: Copy/paste not work in search screen (#1874)

* TW-1825: Fix search results aren't displayed correctly when copy/paste

* TW-1825: Write unit test for this case

* TW-1791: Improve display contacts on multiple homeserver (#1857)

* TW-1791: Refresh contacts when switch another account or login new account

* TW-1791: Write unit test for this case

* TW-1791: Refresh contacts when login or logout

* TW-1791: Fix missing `setUpToMServices` when switch account

* TW-1836: Move shared media to direct chat profile (#1856)

* TW-1836: Add shared media tabs into chat profile

* TW-1836: Delete Shared media screen

* TW-1836: Update `chat_details`

* TW-1869: can not open keyboard when tap on the textfield in IOS (#1877)

* Pinned Nix version to 2.22.1 to fix broken build (#1884)

Also bumped olm to 3.2.16

* TW-1806: fix image weird display on web (#1866)

* TW-1806: add support mime files for web platform, and util method to getTwake supported mimeType

* TW-1806: make image have width and height when the error image when display

* TW-1806: update mime type when drag/drop, paste and add file

* TW-1806: fix wrong display when send HEIC file in android

* TW-1806: display avif file in web

* TW-1806: if image viewer can't be display by real image, use thumbnail

* Tw 1750/improve when upload video (#1875)

* TW-1750: add metadate when sending file event

* TW-1750: fix typo error int videoMimeTypes

* TW-1587: Fix can't open chat when click on notification (#1867)

* TW-1777: Changing greeting message: include tag in greeting message (#1881)

* Bump version to 2.5.9

---------

Co-authored-by: HuyNguyen <[email protected]>
Co-authored-by: --global <[email protected]>
Co-authored-by: Dat PHAM HOANG <[email protected]>
Co-authored-by: sherlock <[email protected]>
Co-authored-by: Nguyen Thai <[email protected]>
Co-authored-by: Bùi Trung Hiếu <[email protected]>
Co-authored-by: KhaledNjim <[email protected]>
Co-authored-by: artembru <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants