[#10519] work week view in dynamic mode
Summary work week view in dynamic mode
Queue Kronolith
Queue Version Git master
Type Enhancement
State Resolved
Priority 1. Low
Owners Jan Schneider <jan (at) horde (dot) org>
Requester rsalmon (at) mbpgroup (dot) com
Created 09/15/11 (244 days ago)
Due
Updated 11/15/11 (183 days ago)
Assigned
Resolved 11/15/11 (183 days ago)
Attachments workweek4.patch Download
Milestone 3.1
Patch Yes

History
11/15/11 14:47:08 rsalmon (at) mbpgroup (dot) com Comment #12 Reply to this comment
Why does it matter which version it will have?
It doesn't. I was just hopping to see the new "work week view" 
function released sooner than April 2012 and make our users happy.

11/15/11 14:19:04 Jan Schneider Comment #11 Reply to this comment
Why does it matter which version it will have?
11/15/11 14:02:31 rsalmon (at) mbpgroup (dot) com Comment #10 Reply to this comment
April 2012.
:-(

Looking at http://www.horde.org/apps/kronolith/roadmap, Kronolith 4.0 
is planned for April 2012

This ticket is set for Milestone 3.1. Is Kronolith 3.1 due for April 
2012 as well ?

Thanks.
11/15/11 13:29:06 Jan Schneider Comment #9 Reply to this comment
April 2012.
11/15/11 08:33:13 rsalmon (at) mbpgroup (dot) com Comment #8 Reply to this comment
Tweaked and committed. Great work, many thanks!
More than happy to contribute ;)

Do you have an approximate time of release for 3.1 ? weeks, months ... 
years  (hope not!)


11/15/11 07:12:04 Jan Schneider Comment #7
Milestone ⇒ 3.1
State ⇒ Resolved
Assigned to Jan Schneider		 Reply to this comment
Tweaked and committed. Great work, many thanks!
11/15/11 06:12:42 Jan Schneider Deleted Attachment: workweek3.patch
 
11/14/11 17:02:56 rsalmon (at) mbpgroup (dot) com Comment #6
New Attachment: workweek4.patch Download		 Reply to this comment
The patch looks really nice, there's only one issue: You can't rely 
on changes in horde.js, because that's a BC break. (Beside that you 
added the new method twice to horde.js).
It should be sufficient to use moveToEndOfWeek() and then move 2 days back.
If you fix this and make sure that the patch applies cleanly to the 
develop branch, we can integrate it. I'm a a bit afraid that this 
might cause space issues in the button bar, but that's a different 
story.
I've updated the patch.


11/14/11 16:14:50 Jan Schneider Comment #5
Patch ⇒ Yes
State ⇒ Feedback		 Reply to this comment
The patch looks really nice, there's only one issue: You can't rely on 
changes in horde.js, because that's a BC break. (Beside that you added 
the new method twice to horde.js).
It should be sufficient to use moveToEndOfWeek() and then move 2 days back.
If you fix this and make sure that the patch applies cleanly to the 
develop branch, we can integrate it. I'm a a bit afraid that this 
might cause space issues in the button bar, but that's a different 
story.
11/14/11 16:05:14 Jan Schneider Deleted Attachment: workweekv2.patch
 
11/14/11 16:05:05 Jan Schneider Deleted Attachment: workweek.patch
 
09/22/11 08:47:05 rsalmon (at) mbpgroup (dot) com Comment #4
New Attachment: workweek3.patch		 Reply to this comment
with the patch attached, everything looks good.

[/var/www/html]# patch -u -p0 -d /var/www/html/horde/ <workweek3.patch

this patch only works with the default theme.


09/15/11 14:30:39 rsalmon (at) mbpgroup (dot) com Comment #3
New Attachment: workweekv2.patch		 Reply to this comment
some typos
09/15/11 13:47:46 rsalmon (at) mbpgroup (dot) com Comment #2
New Attachment: workweek.patch		 Reply to this comment
I've started to work on a patch and it nearly works.

The working week shows 5 days, but events are working with 7 days :-(


09/15/11 13:35:34 rsalmon (at) mbpgroup (dot) com Comment #1
State ⇒ New
Patch ⇒ No
Milestone ⇒
Queue ⇒ Kronolith
Summary ⇒ work week view in dynamic mode
Type ⇒ Enhancement
Priority ⇒ 1. Low		 Reply to this comment
"work week" view is available in traditional mode, but not in dynamic mode.