Opened 8 years ago

Closed 4 years ago

Last modified 4 years ago

#13163 closed enhancement (fixed)

[Mail] use layout api for find window

Reported by: diver Owned by: czeidler
Priority: normal Milestone: R1/beta2
Component: Applications/Mail Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Platform: All

Description

Currently, it doesn't scale well with 20pt. font size.

Attachments (2)

Before.png (59.4 KB ) - added by Janus 7 years ago.
After.png (69.5 KB ) - added by Janus 7 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 by Janus, 7 years ago

Find is different from other applications, it would be nice to use the same behavior of StyledEdit.

  1. There are no replace and find selection menu.
  2. There are no option case-sensitive/loop around etc...
  3. In StyledEdit there is a find window for each StyledEditWindow in Mail there is only one window for all the MailWindow.

I think it is important for the user to have a consistent experience.

Any thoughts?

comment:2 by diver, 7 years ago

Sounds good. Terminal Find should be changed as well. Ideally, it would use Web+ find bar to avoid modal windows.

by Janus, 7 years ago

Attachment: Before.png added

comment:3 by Janus, 7 years ago

patch: 01

by Janus, 7 years ago

Attachment: After.png added

comment:4 by Janus, 7 years ago

Before

After

There is only one problem the ALT-T for "Replace next" collide with "Remove enclosure" command any suggestions?

Last edited 7 years ago by Janus (previous) (diff)

comment:5 by pulkomandy, 7 years ago

patch: 10

comment:6 by pulkomandy, 4 years ago

https://review.haiku-os.org/c/haiku/+/2629

Only conversion to layout kit, I did not implement Janus' proposal.

comment:7 by waddlesplash, 4 years ago

Resolution: fixed
Status: newclosed

Fixed in hrev54169.

comment:8 by nielx, 4 years ago

Assign tickets with status=closed and resolution=fixed within the R1/beta2 development window to the R1/beta2 Milestone

comment:9 by nielx, 4 years ago

Milestone: UnscheduledR1/beta2
Note: See TracTickets for help on using tickets.