Project

General

Profile

Actions

Feature #27096

closed

Mark non-working days in calendar view

Added by Maxim Krušina over 6 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Calendar
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed

Description

In global settings, we can set which are / are not working days - it should be nice to have it visually marked in project's calendar view


Files

Actions #1

Updated by Mischa The Evil over 6 years ago

  • Category set to Calendar
Actions #2

Updated by Toshi MARUYAMA over 6 years ago

  • Description updated (diff)
Actions #3

Updated by Marius BĂLTEANU over 5 years ago

Maybe we can grey-out the non working days using the same style like the days from other months? Something like that:

Actions #4

Updated by Go MAEDA over 5 years ago

Marius BALTEANU wrote:

Maybe we can grey-out the non working days using the same style like the days from other months?

+1
I think it would be even better to use a different background color from dates of previous/next months.

Actions #5

Updated by Marius BĂLTEANU over 5 years ago

  • Assignee set to Marius BĂLTEANU

Go MAEDA wrote:

+1
I think it would be even better to use a different background color from dates of previous/next months.

Yes, it could be an option. Let me see what can I do.

Actions #6

Updated by Marius BĂLTEANU over 5 years ago

Go Maeda, attached the patches that mark the non working days in the calendars from Projects and My Page. I didn't apply any style on the class because I didn't have any idea (apart from the same style as the one from days from other month). Please feel free to add a style to my patches.

Actions #7

Updated by Marius BĂLTEANU about 5 years ago

Go Maeda, did you have the chance to look over this feature?

Actions #8

Updated by Go MAEDA about 5 years ago

  • Assignee changed from Go MAEDA to Marius BĂLTEANU

Marius, thank you for writing the patch.

It looks OK except one thing. Do you think it is OK that the class name for non-working days is different from gantt? Although gantt already uses "nwday", your patch uses "non-working".

Actions #9

Updated by Marius BĂLTEANU about 5 years ago

  • File deleted (0002-Add-non-working-class-to-non-working-days-in-calenda.patch)
Actions #10

Updated by Marius BĂLTEANU about 5 years ago

Go MAEDA wrote:

It looks OK except one thing. Do you think it is OK that the class name for non-working days is different from gantt? Although gantt already uses "nwday", your patch uses "non-working".

For sure we should use the existing "nwday" class, but I didn't know about it. I've updated my second patch, thanks for pointing out.

Tests results here: https://gitlab.com/marius-balteanu/redmine/-/jobs/142133626

Actions #11

Updated by Go MAEDA about 5 years ago

  • Target version changed from Candidate for next major release to 4.1.0

Thank you for updating the patch, it looks good. Setting the target version to 4.1.0.

Actions #12

Updated by Go MAEDA about 5 years ago

  • Status changed from New to Closed
  • Resolution set to Fixed

Committed the patches. Thank you for improving Redmine.

And I have added a style for non-working days. The background color is the same with non-working days in gantt.

Actions #13

Updated by Go MAEDA about 5 years ago

  • Subject changed from Mark (somehow) non-working days in calendar view to Mark non-working days in calendar view
Actions

Also available in: Atom PDF