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 648591 - Account editor window is taller than screen
Account editor window is taller than screen
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Mailer
2.32.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: evolution-mail-maintainers
Evolution QA team
: 653674 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-04-25 09:14 UTC by yang.jie
Modified: 2013-03-31 16:26 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
monitor.png (199.93 KB, image/x-png)
2011-04-25 09:14 UTC, yang.jie
  Details
netbook.png (87.01 KB, image/x-png)
2011-04-25 09:16 UTC, yang.jie
  Details
Wrap the security preferences of an account inside a scrolled window. (60.27 KB, patch)
2011-05-10 08:52 UTC, yang.jie
reviewed Details | Review
patch for latest trunk(based on commit b0e9be5c9ce54c79a26880713ac2617b724e015d) (77.59 KB, patch)
2011-08-02 05:41 UTC, yang.jie
reviewed Details | Review
Account Editor in Evolution 3.5.3 (38.27 KB, image/png)
2012-06-07 11:58 UTC, Matthew Barnes
  Details

Description yang.jie 2011-04-25 09:14:26 UTC
Created attachment 186578 [details]
monitor.png

BUILD IMAGE(X.X.XX.X.XXXXXXXX.X - (e.g.:
meego-netbook-ia32-1.0.90.0.20100831.1)):         
meego-netbook-ia32-1.1.99.0.20110329.5.img
evolution-2.32.1-3.7.i586

HARDWARE MODEL (on what HW this bug is uncovered):
1024*600 netbook
BUG DETAILED DESCRIPTIONS
===========================================================
Add an email account in evolution,when edit the account,found the account
editor window beyond the screen borders.

EXACT STEPS LEADING TO PROBLEM:
(Explain in detail what you do (e.g. tap on OK) and what you see (e.g. message
Connection Failed appears))
===========================================================
1.click mail application and configure an account 
2.click edit option and choose preferences
3.click edit button and account editor window pops up

EXPECTED OUTCOME:
===================
It is displayed in the scope of screen border.
(attached monitor.png)
ACTUAL OUTCOME:
===================
It is beyond the screen border.
(attached netbook.png)

USER IMPACT:
===================

REPRODUCIBILITY:
(always, less than 1/10, 5/10, 9/10)
=====================================
always

EXTRA SOFTWARE INSTALLED:
============================

OTHER COMMENTS:
===================
Comment 1 yang.jie 2011-04-25 09:16:01 UTC
Created attachment 186579 [details]
netbook.png
Comment 2 Akhil Laddha 2011-04-25 10:55:43 UTC
I am not sure if we want to mark it duplicate of bug 267787
Comment 3 yang.jie 2011-04-26 08:22:34 UTC
(In reply to comment #2)
> I am not sure if we want to mark it duplicate of bug 267787

Laddha, I dont think we should mark this duplicate of bug 267787, instead, I think we can make it a blocker of bug 267787.
Comment 4 André Klapper 2011-04-26 11:34:10 UTC
yang.jie: Is there also a MeeGo downstream bug ID to track?
Comment 5 yang.jie 2011-05-03 00:58:06 UTC
(In reply to comment #4)
> yang.jie: Is there also a MeeGo downstream bug ID to track?

yes, bug no: 15023
https://bugs.meego.com/show_bug.cgi?id=15023
Comment 6 André Klapper 2011-05-03 09:15:01 UTC
(In reply to comment #5)
> (In reply to comment #4)
> > yang.jie: Is there also a MeeGo downstream bug ID to track?
> 
> yes, bug no: 15023
> https://bugs.meego.com/show_bug.cgi?id=15023

yang.jie: So please ALWAYS fill out the "See Also" field in BOTH trackers.
Comment 7 yang.jie 2011-05-10 08:52:40 UTC
Created attachment 187565 [details] [review]
Wrap the security preferences of an account inside a scrolled window.

Wrap the security preferences of an account inside a scrolled window.
Comment 8 yang.jie 2011-05-10 08:56:48 UTC
the patch from raul.gutierrez.segales(Wrap the security preferences of an account inside a scrolled window) should have fixed this issue, please help verified and commit, thanks.
Comment 9 Milan Crha 2011-06-29 12:24:46 UTC
Thanks for a bug report and patch. I'm fine to commit it, but it doesn't apply against latest git master of evolution (I'm at commit 3fbb69c, which is before 3.1.3 release). Could you provide a patch for the latest evolution git master, please?
Comment 10 Matthew Barnes 2011-06-29 23:09:07 UTC
*** Bug 653674 has been marked as a duplicate of this bug. ***
Comment 11 Akhil Laddha 2011-07-26 08:12:10 UTC
yang.jie, ping, can you please provide patch against master (3.1.5) source before it gets too later for stable 3.2.0 release ?
Comment 12 yang.jie 2011-08-02 05:41:25 UTC
Created attachment 193027 [details] [review]
patch for latest trunk(based on commit b0e9be5c9ce54c79a26880713ac2617b724e015d)

patch for latest trunk(based on commit b0e9be5c9ce54c79a26880713ac2617b724e015d).
Comment 13 yang.jie 2011-08-02 05:44:00 UTC
(In reply to comment #11)
> yang.jie, ping, can you please provide patch against master (3.1.5) source
> before it gets too later for stable 3.2.0 release ?

please check attached patch file on comment #12.
Comment 14 Milan Crha 2011-08-02 14:44:05 UTC
Thanks for the update, this applies cleanly. The only thing is that it doesn't work always. It works fine for POP accounts, but does not work for IMAP accounts, which have too many Receiving Options.

What I would like also to see, is to make the window smaller only if the screen size is not sufficient. Because my screen size is tall enough to show me whole Security tab in once, but with the patch (and POP account) it shows me only portion of it, with a scroll bar. It doesn't look good.
Comment 15 Burt Silverman 2011-08-26 15:03:02 UTC
No new information. Just reporting that I am experiencing the issue, too. Looking forward to a fix.
Comment 16 Milan Crha 2012-06-06 11:24:22 UTC
Matthew, is this addressed for 3.5.3?
Comment 17 Matthew Barnes 2012-06-06 11:36:08 UTC
Depends on the screen size and the account type, of course.  IMAP usually being the worst offender, I am going to remove from the UI some of the more esoteric options like shell command and number of connections.
Comment 18 André Klapper 2012-06-07 11:45:34 UTC
@mbarnes: In case you are aware of UI changes in the 3.5 Account Editor please let me know so I can check the User Docs. I prefer to avoid checking all 12 (or so) account types with all their tabs... :)
Comment 19 Matthew Barnes 2012-06-07 11:58:36 UTC
Created attachment 215845 [details]
Account Editor in Evolution 3.5.3

(In reply to comment #18)
> @mbarnes: In case you are aware of UI changes in the 3.5 Account Editor please
> let me know so I can check the User Docs. I prefer to avoid checking all 12 (or
> so) account types with all their tabs... :)

I rewrote the account editor from scratch for 3.5.3.  Options are the same but you'll want to look it over.  The attached screenshot is what editing an account looks like now.

Once I get regressions under control, I'm going go through the User Docs more thoroughly and file bugs for parts that need changed.  It's a different world now with the account-mgmt merge.
Comment 20 Milan Crha 2012-07-27 06:17:16 UTC
Downstream bug report about the same in 3.4.3 (not express mode only):
https://bugzilla.redhat.com/show_bug.cgi?id=843665
Comment 21 Matthew Barnes 2013-03-31 16:26:31 UTC
This got addressed in the Account Editor rewrite for Evolution 3.6.

Closing as FIXED.