Ticket #3179 (closed defect: fixed)
"Search string not found" dialog inconsistencies
| Reported by: | egmont | Owned by: | andrew_b |
|---|---|---|---|
| Priority: | trivial | Milestone: | 4.8.12 |
| Component: | mc-core | Version: | master |
| Keywords: | Cc: | ||
| Blocked By: | Blocking: | ||
| Branch state: | merged | Votes for changeset: | committed-master |
Description
Search for a string (with F7) that's not found.
In mcview, the dialog has the colors of a normal dialog, with whitespace above/below the "Search string not found" message and no button. Pressing any key closes the dialog.
In mcedit and in the diffviewer, the dialog has the colors of an error box, no vertical whitespace, but a separator line and a Dismiss button. Only space or enter close the dialog.
I believe these two should look and behave the same, or at least closer to each other than they currently do. I vote for normal dialog colors, I don't think it's an error if you search for something and it's not found. For the rest, I leave it up to you :)
Change History
comment:1 in reply to: ↑ description Changed 11 years ago by andrew_b
- Status changed from new to accepted
- Owner set to andrew_b
- Branch state changed from no branch to on review
- Milestone changed from Future Releases to 4.8.12
comment:2 Changed 11 years ago by slavazanko
- Votes for changeset set to slavazanko
- Branch state changed from on review to approved
comment:3 Changed 11 years ago by andrew_b
- Status changed from accepted to testing
- Votes for changeset changed from slavazanko to committed-master
- Resolution set to fixed
- Branch state changed from approved to merged
Merged to master: [c8eb0f4c965b3664b2f49d7d7baebba60b34fd91].
Note: See
TracTickets for help on using
tickets.
Replying to egmont:
I agree.
Branch:3179_not_found_dialog_uniq
changeset:a787608898b053b8d15a86084bb3b0cae1ec4af9