feature request #7717
closedImplement "continued" in TimePeriod UI
0%
Description
The continued flag should be alternative to the end date. As described in #7644 either the flag is set or we have an end date. So setting the flag in the TimePeriod class actually removes the end date.
Therefore, when the flag is set it should not be possible to edit the end date.
Related issues
Updated by Andreas Müller almost 6 years ago
- Due date set to 08/15/2018
- Start date changed from 09/04/2018 to 08/15/2018
- Follows feature request #7644: Add "continued" flag to (Verbatim)Timeperiod added
Updated by Andreas Müller almost 6 years ago
- Description updated (diff)
- Due date deleted (
08/15/2018) - Priority changed from New to Highest
- Start date changed from 08/15/2018 to 09/04/2018
Updated by Andreas Müller almost 6 years ago
for the expected behavior see also TimePeriodTest.testContinued and .testToStringTimePeriod
Updated by Katja Luther almost 6 years ago
- Status changed from New to Resolved
- % Done changed from 0 to 50
Applied in changeset taxeditor|f36a68ee91d7e3c3c8365b13507c25af4da48b81.
Updated by Katja Luther almost 6 years ago
- Status changed from Resolved to New
- Assignee changed from Katja Luther to Andreas Müller
- % Done changed from 50 to 0
the continued flag is implemented in editor now. at the moment the end date is removed when the continued flag is set to avoid confusion when the end date is displayed in the ui but is not saved.
Updated by Andreas Müller almost 6 years ago
I don't know if this is related, but when entering data into the parse field now, the cursor always jumps to the beginning. This is very critical.
Updated by Andreas Müller almost 6 years ago
Also I would move the continued flag up, before "End" as it is something inbetween start and end. This may make the behavior a bit clearer.
Can we also grey out the labels of the "end line" this makes it clearer from the beginning that "end" is not available at current state (minor request).
Updated by Andreas Müller almost 6 years ago
Best would be to not show the "end" line at all if continued == true
Updated by Andreas Müller almost 6 years ago
- Status changed from Resolved to Feedback
- Assignee changed from Andreas Müller to Katja Luther
Updated by Katja Luther almost 6 years ago
- Status changed from Feedback to Resolved
- % Done changed from 0 to 50
Applied in changeset taxeditor|a1fbc94af7ad08a851eb87b92e7b8aa34b1f6a89.
Updated by Katja Luther almost 6 years ago
- % Done changed from 50 to 0
Andreas Müller wrote:
Best would be to not show the "end" line at all if continued == true
I think it is clearer to show the disabled line to show that it is possible to enter a end date.
Updated by Katja Luther almost 6 years ago
- Assignee changed from Katja Luther to Andreas Müller
Updated by Andreas Müller almost 6 years ago
- Status changed from Resolved to Closed
- Assignee changed from Andreas Müller to Katja Luther
It works well now and is also intuitive IMO.