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-1456: Button new chat doesn t work #1596

Merged
merged 6 commits into from
Mar 29, 2024
Merged

Conversation

Te-Z
Copy link
Contributor

@Te-Z Te-Z commented Mar 16, 2024

Issue: #1456

Resolution:

Capture.video.du.28-03-2024.15.19.59.webm
Capture.video.du.28-03-2024.15.31.53.webm

Copy link

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

@sherlockvn
Copy link
Collaborator

Also test case responsive: tablet and mobile in web

@hoangdat
Copy link
Member

image

Ripple shape is not so beutifull

  • padding for it
  • round corner for shape

or only take the ripple for the Copy button.

@Te-Z Te-Z force-pushed the TW-1456/button-new-doesn-t-work branch from f93113c to 689d4aa Compare March 18, 2024 10:01
@Te-Z
Copy link
Contributor Author

Te-Z commented Mar 18, 2024

image Ripple shape is not so beutifull
  • padding for it
  • round corner for shape

or only take the ripple for the Copy button.

Updated @hoangdat . WDYT ?

Capture.video.du.18-03-2024.10.46.37.webm

@Te-Z
Copy link
Contributor Author

Te-Z commented Mar 18, 2024

Also test case responsive: tablet and mobile in web

There you go @sherlockvn

Capture.video.du.18-03-2024.10.56.58.webm

@Te-Z Te-Z force-pushed the TW-1456/button-new-doesn-t-work branch from 689d4aa to 6ac7703 Compare March 18, 2024 13:58
@Te-Z Te-Z force-pushed the TW-1456/button-new-doesn-t-work branch from daab3b3 to f6cd9ac Compare March 28, 2024 14:44
@Te-Z
Copy link
Contributor Author

Te-Z commented Mar 28, 2024

Issue: #1456

Resolution:

Capture.video.du.28-03-2024.15.19.59.webm
Capture.video.du.28-03-2024.15.31.53.webm

demonstration videos updated
I alse reverted active id changes to handle this case sin an other pull request

@Te-Z Te-Z force-pushed the TW-1456/button-new-doesn-t-work branch from f6cd9ac to dbd04f6 Compare March 28, 2024 15:04
@Te-Z Te-Z force-pushed the TW-1456/button-new-doesn-t-work branch from dbd04f6 to 8a44b28 Compare March 28, 2024 15:11
@Te-Z Te-Z force-pushed the TW-1456/button-new-doesn-t-work branch from abca529 to 8e6f26a Compare March 28, 2024 15:20
@hoangdat
Copy link
Member

Hi @Te-Z , back button in profile info still not go to chat details

Screen.Recording.2024-03-28.at.22.23.59.mov

@Te-Z Te-Z force-pushed the TW-1456/button-new-doesn-t-work branch from 8e6f26a to 1fb8307 Compare March 28, 2024 15:25
@Te-Z Te-Z force-pushed the TW-1456/button-new-doesn-t-work branch from 1fb8307 to 6ae0b91 Compare March 28, 2024 16:09
@Te-Z
Copy link
Contributor Author

Te-Z commented Mar 28, 2024

Hi @Te-Z , back button in profile info still not go to chat details

Screen.Recording.2024-03-28.at.22.23.59.mov

@hoangdat fixed. I use the same logic used to open the add members screen to open the profile info screen from the bottom sheet.

Capture.video.du.28-03-2024.17.11.14.webm

@hoangdat hoangdat merged commit fecf450 into main Mar 29, 2024
3 checks passed
@hoangdat hoangdat deleted the TW-1456/button-new-doesn-t-work branch March 29, 2024 03:44
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.

5 participants