After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 628005 - Russian: evolution <Advanced Search> dialogs are oversized
Russian: evolution <Advanced Search> dialogs are oversized
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: general
2.28.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Evolution Shell Maintainers Team
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2010-08-26 07:03 UTC by hao.h.li
Modified: 2012-10-27 19:45 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
fedora12_ru.png (41.91 KB, image/png)
2010-08-26 07:04 UTC, hao.h.li
  Details
fedora12_enu.png (33.69 KB, image/png)
2010-08-26 07:04 UTC, hao.h.li
  Details
628005.patch (3.52 KB, patch)
2010-08-26 07:13 UTC, hao.h.li
needs-work Details | Review
628005.patch (4.16 KB, patch)
2010-08-30 05:12 UTC, hao.h.li
none Details | Review
screen shot from the final commit (32.01 KB, image/jpeg)
2010-08-30 09:44 UTC, Chenthill P
  Details

Description hao.h.li 2010-08-26 07:03:36 UTC
To reproduce <Advanced Search> dialog:
1. Run evolution in Russian language
2. Go To Search menu > Click Advanced Search
Then Notice the issue (See attachment-fedora12_ru.png), compared with attachment-fedora12_eng.png, the <Advanced Search> dialog in ru was oversized.
Comment 1 hao.h.li 2010-08-26 07:04:14 UTC
Created attachment 168781 [details]
fedora12_ru.png
Comment 2 hao.h.li 2010-08-26 07:04:43 UTC
Created attachment 168782 [details]
fedora12_enu.png
Comment 3 hao.h.li 2010-08-26 07:13:28 UTC
Created attachment 168784 [details] [review]
628005.patch

I created the patch in attachment. I just tried to move "Include threads" label with is combox to new line, so the UI in Russian may not be oversized in horizontal. 

Could you review if the patch is OK or you may have better way to fix it? 
If it is OK, please help to check-in the patch into master branch and then I will submit it to MeeGo branch to make these 2 branch be same.

Thanks!
Comment 4 Chenthill P 2010-08-29 17:00:34 UTC
It would be better if the 'Add condition' button appears below 'Include threads' as its associated with tree view where the conditions are displayed. So the order would be,
Find items:
Include threads
Add condition
<Tree View>

This patch cannot go into master as we are current under UI freeze. Please provide an updated patch and it would be committed once we branch master for evolution-2.32 release.
Comment 5 Chenthill P 2010-08-29 17:01:47 UTC
Review of attachment 168784 [details] [review]:

marking as needs-work.
Comment 6 Chenthill P 2010-08-29 17:04:50 UTC
Though this is a UI change, it could be considered as a usability fix as well. Please attach the patch and I will inquire if its possible to absorb this patch for 2.32 release..
Comment 7 hao.h.li 2010-08-30 05:12:36 UTC
Created attachment 169031 [details] [review]
628005.patch

Updated the patch to make the order be
Find items:
Include threads
Add condition
<Tree View>

Could you check it? Thanks!
Comment 8 Chenthill P 2010-08-30 09:43:06 UTC
I would consider this as a usability fix. Added additional padding of 12 pixels to make the dialog look better and commit the patch to master branch.

commit id - 252ed73211d3e89077a7718e857a11a6f41e4e05
Comment 9 Chenthill P 2010-08-30 09:44:41 UTC
Created attachment 169040 [details]
screen shot from the final commit
Comment 10 Matthew Barnes 2010-08-30 10:32:01 UTC
We should turn the Search Folder Sources options into a combo box to free up more vertical space for the actual search conditions.  The box looks pretty squished.  Can be post-2.32.
Comment 11 André Klapper 2012-06-20 12:48:22 UTC
So is there anything left here to fix?
Current situation seems to be acceptable.
Comment 12 Tobias Mueller 2012-10-27 19:45:19 UTC
So I assume this is FIXED then. Please reopen if this is not the case.