6.0.0-alpha14
7/2/25

[#11262] IMP UI issues
Summary IMP UI issues
Queue IMP
Queue Version Git develop
Type Bug
State Resolved
Priority 3. High
Owners Horde Developers (at) , jan (at) horde (dot) org, slusarz (at) horde (dot) org
Requester slusarz (at) horde (dot) org
Created 07/10/2012 (4740 days ago)
Due
Updated 09/06/2017 (2856 days ago)
Assigned
Resolved 10/02/2012 (4656 days ago)
Github Issue Link
Github Pull Request
Milestone 6
Patch No

History
09/06/2017 08:18:16 AM Git Commit Comment #20 Reply to this comment
Changes have been made in Git (master):

commit 49e9c07d385992b42e57b733188a5fdc63411823
Author: Justin Patrin <justinpatrin@php.net>
Date:   Sat Jun 9 23:11:25 2007 +0000

     Bug #11262 Fix issues with any <code> block causing no output


     git-svn-id: 
https://svn.php.net/repository/pear/packages/Text_Wiki/trunk@237313 
c90b9560-bf6c-de11-be94-00142212c4b1

  Text/Wiki/Parse/Default/Code.php | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

http://github.com/horde/horde/commit/49e9c07d385992b42e57b733188a5fdc63411823
10/02/2012 12:36:41 AM Michael Slusarz Comment #19
State ⇒ Resolved
Reply to this comment
AFAICT, the issues in this ticket have, to one degree or another, been 
addressed.  Closing ticket.
08/29/2012 12:48:42 PM Git Commit Comment #18 Reply to this comment
Changes have been made in Git (master):

commit 7fca9c1d279336bf28364e04c3e21899979f8cd0
Author: Jan Schneider <jan@horde.org>
Date:   Tue Aug 21 18:39:27 2012 +0200

     Make buttons active on hover (Bug #11262).

  framework/Core/js/spellchecker.js                  |    4 +-
  horde/themes/default/graphics/button-active-bg.png |  Bin 0 -> 2812 bytes
  horde/themes/default/graphics/buttonbar-bg.png     |  Bin 2819 -> 2824 bytes
  horde/themes/default/graphics/tablehead-bg.png     |  Bin 122 -> 149 bytes
  horde/themes/default/screen.css                    |   72 ++++++++++++++-----
  imp/lib/Ui/Compose.php                             |   10 ++-
  imp/templates/dynamic/compose.html.php             |   10 ++--
  imp/templates/dynamic/mailbox.html.php             |   18 +++---
  imp/templates/dynamic/message.html.php             |   10 ++--
  imp/templates/dynamic/redirect.html.php            |    3 +-
  imp/templates/imp/mailbox/header.html              |   12 ++--
  imp/templates/imp/mailbox/navbar.html              |   17 +++--
  imp/themes/default/dynamic/screen.css              |    3 +
  imp/themes/default/graphics/popdown.png            |  Bin 123 -> 3040 bytes
  imp/themes/default/screen.css                      |   18 +++--
  15 files changed, 113 insertions(+), 64 deletions(-)

http://git.horde.org/horde-git/-/commit/7fca9c1d279336bf28364e04c3e21899979f8cd0
08/29/2012 12:46:13 PM Git Commit Comment #17 Reply to this comment
Changes have been made in Git (master):

commit a96460170d3829c4751817d03e1e7f21020b6643
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Wed Aug 1 15:49:32 2012 -0600

     [mms] Fix sidebar element wrapping (Ticket #11262).

  horde/docs/CHANGES                    |    1 +
  horde/package.xml                     |    2 +
  horde/templates/sidebar/_row.html.php |   12 +++--
  horde/templates/tree/_row.html.php    |   21 ++++----
  horde/themes/default/screen.css       |   84 +++++++++++++-------------------
  imp/docs/CHANGES                      |    1 +
  imp/js/dimpbase.js                    |   21 ++++----
  imp/js/dimpcore.js                    |    2 -
  imp/lib/Dynamic/Mailbox.php           |    1 -
  imp/package.xml                       |    2 +
  imp/themes/default/dynamic/screen.css |    8 +++
  11 files changed, 75 insertions(+), 80 deletions(-)

http://git.horde.org/horde-git/-/commit/a96460170d3829c4751817d03e1e7f21020b6643
08/29/2012 12:42:43 PM Git Commit Comment #16 Reply to this comment
Changes have been made in Git (master):

commit 73454fff08873d6620a6de2ea13271fc2230e797
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Mon Jul 16 22:16:57 2012 -0600

     Re-add indication that preview can be toggled (Request #11262).

  imp/templates/dynamic/mailbox.html.php |    6 ++++--
  imp/themes/default/dynamic/screen.css  |   13 +++++++------
  2 files changed, 11 insertions(+), 8 deletions(-)

http://git.horde.org/horde-git/-/commit/73454fff08873d6620a6de2ea13271fc2230e797
08/29/2012 12:42:32 PM Git Commit Comment #15 Reply to this comment
Changes have been made in Git (master):

commit e7a3792a3a5cae14c52a086f297751eadf03d2cd
Author: Jan Schneider <jan@horde.org>
Date:   Mon Jul 16 17:03:48 2012 +0200

     Add animated checkmail icon (Bug #11262).

  horde/themes/default/graphics/reload.png |  Bin 615 -> 911 bytes
  imp/js/dimpbase.js                       |    5 ++---
  imp/themes/default/dynamic/screen.css    |    3 +++
  imp/themes/default/graphics/reload.gif   |  Bin 0 -> 13593 bytes
  imp/themes/default/graphics/reload.png   |  Bin 615 -> 911 bytes
  5 files changed, 5 insertions(+), 3 deletions(-)

http://git.horde.org/horde-git/-/commit/e7a3792a3a5cae14c52a086f297751eadf03d2cd
08/29/2012 12:41:40 PM Git Commit Comment #14 Reply to this comment
Changes have been made in Git (master):

commit 2150fa7738e9175b20943981ef220b1a0bacd40e
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Wed Jul 11 00:30:03 2012 -0600

     Fix subfolder indentation (Bug #11262)

  horde/themes/default/screen.css       |    3 ---
  imp/themes/default/dynamic/screen.css |    4 ++++
  2 files changed, 4 insertions(+), 3 deletions(-)

http://git.horde.org/horde-git/-/commit/2150fa7738e9175b20943981ef220b1a0bacd40e
08/22/2012 02:11:49 PM Git Commit Comment #13 Reply to this comment
Changes have been made in Git (develop):

commit 7fca9c1d279336bf28364e04c3e21899979f8cd0
Author: Jan Schneider <jan@horde.org>
Date:   Tue Aug 21 18:39:27 2012 +0200

     Make buttons active on hover (Bug #11262).

  framework/Core/js/spellchecker.js                  |    4 +-
  horde/themes/default/graphics/button-active-bg.png |  Bin 0 -> 2812 bytes
  horde/themes/default/graphics/buttonbar-bg.png     |  Bin 2819 -> 2824 bytes
  horde/themes/default/graphics/tablehead-bg.png     |  Bin 122 -> 149 bytes
  horde/themes/default/screen.css                    |   72 ++++++++++++++-----
  imp/lib/Ui/Compose.php                             |   10 ++-
  imp/templates/dynamic/compose.html.php             |   10 ++--
  imp/templates/dynamic/mailbox.html.php             |   18 +++---
  imp/templates/dynamic/message.html.php             |   10 ++--
  imp/templates/dynamic/redirect.html.php            |    3 +-
  imp/templates/imp/mailbox/header.html              |   12 ++--
  imp/templates/imp/mailbox/navbar.html              |   17 +++--
  imp/themes/default/dynamic/screen.css              |    3 +
  imp/themes/default/graphics/popdown.png            |  Bin 123 -> 3040 bytes
  imp/themes/default/screen.css                      |   18 +++--
  15 files changed, 113 insertions(+), 64 deletions(-)

http://git.horde.org/horde-git/-/commit/7fca9c1d279336bf28364e04c3e21899979f8cd0
08/13/2012 10:45:05 PM Michael Slusarz Comment #12 Reply to this comment
* Double separator lines in context menus.
I don't really care about this as much.  It is an acceptable solution for now.
08/13/2012 01:47:23 PM Jan Schneider Comment #11 Reply to this comment
Current state:

* Double separator lines in context menus.

* Action buttons are broken - or at least are not intuitive since they
require clicking on the text.  Clicking the surrounding space/icon
doesn't work.  There needs to be a definable area to click on -
namely, all space inside of the vertical separators.

* There is no feedback when clicking/hovering action buttons. (Jan
says he has not yet ported from delivered calendar design)

* From/To/Cc/Bcc names need to be distinguished somehow.  I propose
either underlines or background shading. Jan wants to do popdown
icons, but I feel that is too intrusive and, more important, does not
deal with the issue of grouping the whole e-mail name together.

* Popdown menus should automatically trigger on icon hover.  For
something like reply/forward, the popdown menu should automatically be
triggered when hovering over the popdown icon.  This is different from
the topmenu, since there the popdown menu is triggered when hovering
over the main text.
08/13/2012 01:37:16 PM Jan Schneider Comment #10 Reply to this comment
* Folder lists: long folder names break the tree.  Need to hide long 
names instead of wrapping.

* Moving mailboxes in folder list.
I think those 2 are fixed already too?
08/13/2012 01:33:41 PM Jan Schneider Comment #9 Reply to this comment
* If possible, expand/collapse folder icons need to be improved.
This is done already.
07/17/2012 04:37:53 AM Michael Slusarz Comment #8 Reply to this comment
* There is no feedback when clicking/hovering refresh that something 
is actually happening.
Fixed.
* There's also no discernible indication that the preview header can 
be toggled.
Fixed.
07/17/2012 04:35:58 AM Git Commit Comment #7 Reply to this comment
Changes have been made in Git (develop):

commit 73454fff08873d6620a6de2ea13271fc2230e797
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Mon Jul 16 22:16:57 2012 -0600

     Re-add indication that preview can be toggled (Request #11262).

  imp/templates/dynamic/mailbox.html.php |    6 ++++--
  imp/themes/default/dynamic/screen.css  |   13 +++++++------
  2 files changed, 11 insertions(+), 8 deletions(-)

http://git.horde.org/horde-git/-/commit/73454fff08873d6620a6de2ea13271fc2230e797
07/16/2012 03:04:40 PM Git Commit Comment #6 Reply to this comment
Changes have been made in Git (develop):

commit e7a3792a3a5cae14c52a086f297751eadf03d2cd
Author: Jan Schneider <jan@horde.org>
Date:   Mon Jul 16 17:03:48 2012 +0200

     Add animated checkmail icon (Bug #11262).

  horde/themes/default/graphics/reload.png |  Bin 615 -> 911 bytes
  imp/js/dimpbase.js                       |    5 ++---
  imp/themes/default/dynamic/screen.css    |    3 +++
  imp/themes/default/graphics/reload.gif   |  Bin 0 -> 13593 bytes
  imp/themes/default/graphics/reload.png   |  Bin 615 -> 911 bytes
  5 files changed, 5 insertions(+), 3 deletions(-)

http://git.horde.org/horde-git/-/commit/e7a3792a3a5cae14c52a086f297751eadf03d2cd
07/13/2012 12:01:47 AM Jan Schneider Comment #5 Reply to this comment
* Message flag icons need to be reduced in size (Jan will look into this)
This is done.
* There is no mailbox size/message location anymore in dynamic view.
Done too.
07/11/2012 07:41:26 AM Michael Slusarz Comment #4 Reply to this comment
* Message screen: There are double popdowns for reply/forward.
Fixed in Ticket #11257.
07/11/2012 07:35:43 AM Michael Slusarz Comment #3 Reply to this comment
* Folder lists are broken: everything after the first child level is 
displayed at the same indentation.
This has been fixed.

Additional issue:  There is currently no indication on the screen of 
the current mailbox name.  We can't use the title, since it may not be 
available.  And you can't use the folder tree since it may not be 
visible (scrolled off screen).
07/11/2012 06:40:04 AM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (develop):

commit 2150fa7738e9175b20943981ef220b1a0bacd40e
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Wed Jul 11 00:30:03 2012 -0600

     Fix subfolder indentation (Bug #11262)

  horde/themes/default/screen.css       |    3 ---
  imp/themes/default/dynamic/screen.css |    4 ++++
  2 files changed, 4 insertions(+), 3 deletions(-)

http://git.horde.org/horde-git/-/commit/2150fa7738e9175b20943981ef220b1a0bacd40e
07/10/2012 09:19:55 PM Michael Slusarz Comment #1
Patch ⇒ No
State ⇒ Assigned
Milestone ⇒ 6
Assigned to Horde DevelopersHorde Developers
Assigned to Michael Slusarz
Queue ⇒ IMP
Priority ⇒ 3. High
Type ⇒ Bug
Summary ⇒ IMP UI issues
Assigned to Jan Schneider
Reply to this comment
UI issues that need to be addressed before 6.0 is released:

* Message flag icons need to be reduced in size (Jan will look into this)

* Folder lists are broken: everything after the first child level is 
displayed at the same indentation.

* If possible, expand/collapse folder icons need to be improved.

* Double separator lines in context menus.

* Action buttons are broken - or at least are not intuitive since they 
require clicking on the text.  Clicking the surrounding space/icon 
doesn't work.  There needs to be a definable area to click on - 
namely, all space inside of the vertical separators.

* There is no feedback when clicking/hovering action buttons. (Jan 
says he has not yet ported from delivered calendar design)

* There is no feedback when clicking/hovering refresh that something 
is actually happening.

* There's also no discernible indication that the preview header can 
be toggled.

* From/To/Cc/Bcc names need to be distinguished somehow.  I propose 
either underlines or background shading. Jan wants to do popdown 
icons, but I feel that is too intrusive and, more important, does not 
deal with the issue of grouping the whole e-mail name together.

* Popdown menus should automatically trigger on icon hover.  For 
something like reply/forward, the popdown menu should automatically be 
triggered when hovering over the popdown icon.  This is different from 
the topmenu, since there the popdown menu is triggered when hovering 
over the main text.

* There is no mailbox size/message location anymore in dynamic view.

* Folder lists: long folder names break the tree.  Need to hide long 
names instead of wrapping.

* Message screen: There are double popdowns for reply/forward.

* Moving mailboxes in folder list.

Saved Queries