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.

Sign up for GitHub

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

Jump to bottom

TF-2948 Apply new identity view #3322

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

TF-2948 Apply new identity view #3322

wants to merge 23 commits into from

Conversation

dab246
Copy link
Member

@dab246 dab246 commented Dec 4, 2024

Issue

#2948

Resolved

  • Web:
Screen.Recording.2024-12-04.at.16.36.18.mov
  • Mobile:
demo-android.webm

dab246 and others added 22 commits November 25, 2024 18:24
@dab246
TF-3298 Fix blink when refresh email list (#3299)
d339dd8
@tddang-linagora @hoangdat
Update TMail backend docker to memory-1.0.0
e50a23d
@tddang-linagora @hoangdat
TF-3265 Fix double scrolling composer
1a58542
@dab246 @hoangdat
TF-3291 Fix [SEARCH] No results if you add > 1 address in the From field
c475735
@tddang-linagora
TF-3264 Fix composer drag drop all modes (#3279)
2128a6e
@tddang-linagora
Delegate cache control from Flutter to browser (#3289)
9c3954a
@dab246 @hoangdat
TF-3292 Fix [SEARCH] If I filter emails by date and then sort them by…
eaee8a0 
… relevance, the filter isn't applied
@dab246 @hoangdat
TF-3292 Write integration test for search emails by date and then sor…
e59fc1f 
…t them by relevance
@dab246 @hoangdat
TF-3294 Fix BLUE-BAR mail to attendees duplicated recipients
c5b1e87
@dab246 @hoangdat
TF-3295 Fix BLUE-BAR attendees mail addresses are not clickable
6fff59a
@dab246 @hoangdat
TF-3296 Fix [SEARCH] Can't apply the "Starred" filter in search
3a5f91b
@dab246 @hoangdat
fixup! TF-3296 Fix [SEARCH] Can't apply the "Starred" filter in search
9202925
@dab246 @hoangdat
TF-3315 Fix TMail web could not display embedded table correctly
fe2062d
@florentos17 @hoangdat
TF-3189 new option to enable/disable subaddressing for a personal folder
ed0b14c
@florentos17 @hoangdat
TF-3189 new option to copy a folder's subaddress
810e304
@florentos17 @hoangdat
TF-3189 subaddressing features only shown if supported by the server
0f61e3d
@florentos17 @hoangdat
TF-3189 new confirmation popup when enabling subaddressing for a folder
d939ffb
@florentos17 @hoangdat
TF-3189 new reply to field in the mail composer
20c4720
@florentos17 @hoangdat
TF-3189 composer now correctly encodes subaddresses
47b842d
@dab246
TF-2948 Apply new identity view for web
2ae09ad
@dab246
TF-2948 Apply new identity view for mobile
1654db1
@dab246
TF-2948 Set identity as default
4bc211c
@dab246 dab246 requested a review from hoangdat December 4, 2024 09:39
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 4, 2024

This PR has been deployed to https://linagora.github.io/tmail-flutter/3322.

tddang-linagora
tddang-linagora previously approved these changes Dec 5, 2024
View reviewed changes
@hoangdat
Copy link
Member

  • it is nested scroll in this screen (both mobile/web), it makes user sometime hared to scroll, especially in mobile. So I propose to disable scroll for preview signature, fixed height for this. WDYT?
    image

@hoangdat
Copy link
Member

  • Create new profile button is moving up when identity list loading. Should keep it pins to bottom of screen
image

@hoangdat hoangdat dismissed tddang-linagora’s stale review February 5, 2025 11:06

The merge-base changed after approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tddang-linagora tddang-linagora tddang-linagora left review comments

@hoangdat hoangdat Awaiting requested review from hoangdat

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@dab246 @hoangdat @tddang-linagora @florentos17