Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Introduce year grid view (with FC multiMonthYear plugin) #5248

Merged
merged 1 commit into from
Jul 10, 2023

Conversation

tcitworld
Copy link
Member

@tcitworld tcitworld commented May 21, 2023

Closes #159

image

Note: Shortcut for ListView is changed from 4 to 5 for consistency.

It could use a little style enhancement.

FullCalendar provides two "year" views: this one (multiMonthYear) and another one (dayGridYear). This one seems more suitable.

@tcitworld tcitworld self-assigned this May 21, 2023
@tcitworld tcitworld changed the title feat(view): Introduce year grid view (with FC multiMonthYear plugin) Introduce year grid view (with FC multiMonthYear plugin) May 21, 2023
@codecov
Copy link

codecov bot commented May 21, 2023

Codecov Report

Patch coverage has no change and project coverage change: +42.72 🎉

Comparison is base (1e9a34d) 22.78% compared to head (e4ab210) 65.50%.

Additional details and impacted files
@@              Coverage Diff              @@
##               main    #5248       +/-   ##
=============================================
+ Coverage     22.78%   65.50%   +42.72%     
  Complexity      371      371               
=============================================
  Files           237       36      -201     
  Lines         11636     1986     -9650     
  Branches       2255        0     -2255     
=============================================
- Hits           2651     1301     -1350     
+ Misses         8985      685     -8300     
Flag Coverage Δ
javascript ?
php 65.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 201 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@ChristophWurst ChristophWurst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks clean

@@ -64,7 +65,7 @@ import {
modifyDate,
} from '../../../utils/date.js'
import { mapState } from 'vuex'
import formatDateRage from '../../../filters/dateRangeFormat.js'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ngl working with dates sometimes ends in rage

@ChristophWurst ChristophWurst mentioned this pull request May 22, 2023
@miaulalala
Copy link
Contributor

needs a rebase, but other than that

image

@tcitworld
Copy link
Member Author

rebased

@jean-gui jean-gui removed their request for review June 27, 2023 13:55
@miaulalala miaulalala added this to the v4.5.0 milestone Jul 4, 2023
@miaulalala
Copy link
Contributor

Sorry, can you rebase once more? Static Analysis hasn't run because of timing.

Closes #159

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
@miaulalala miaulalala merged commit 33a26c1 into main Jul 10, 2023
@miaulalala miaulalala deleted the feat-year-view branch July 10, 2023 12:32
@bernd-wechner
Copy link

Awesome! Cannot wait to see this. Whenever possible let us know which version it appears in (nominally 4.5.0 of the Calendar app). Job well done! Been waiting on this for years.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

year view
4 participants