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

Wallcalendar #298

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Wallcalendar #298

wants to merge 3 commits into from

Conversation

howardgrigg
Copy link

@howardgrigg howardgrigg commented Dec 15, 2023

First go at the WallCalendar module - needs review and documentation.

WallCalendar displays a grid of upcoming weeks and writes the event titles in each of the grid squares for each day.

Currently it doesn't work with other modules as the image is rotated 90 degrees for landscape orientation.

IMG_3486

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hi there and welcome to Inkycal. Thanks for opening this issue. As this is your first Pull-Request in this repository, please read through the contributing guidelines. Please note that non-critical pull-request cannot be merged into the main branch to ensure stability. Please create a new branch and ask to have it merged into main. Thanks for your understanding.

@aceisace
Copy link
Member

Thanks for creating this PR @howardgrigg . Let me know once this is ready for reviewing so I can integrate it 👍

@aceisace aceisace self-assigned this Jan 14, 2024
@aceisace aceisace added the new feature request For possible new features label Jan 14, 2024
@aceisace
Copy link
Member

aceisace commented Feb 9, 2024

@howardgrigg May I kindly ask you to use the default fonts already provided by Inkycal, namely NotoSans? The reason is that most fonts generally only support very few languages and attempting to show letters not supported by the font will result in tofus instead of the actual character, which may be confusing for users or even considered as a bug. The additional fonts can then be removed from this PR, but you may put a note in the module of your preferred font 👍

@howardgrigg
Copy link
Author

Yea sure, I've got a few edits as well that I've made over the past month - odd little bugs that have popped up, mainly around line length issues so I'll include those changes as well. Hopefully get to it in the next couple of days.

@aceisace
Copy link
Member

Sure thing, thanks for the update!

Copy link

Marking this PR as stale due to inactivity

@J-CMartin
Copy link

@howardgrigg Is this something you will be updating? I'm very interested on a calendar with that format. It looks really good and it's what I'm looking for

Copy link

Marking this PR as stale due to inactivity

Copy link

Marking this PR as stale due to inactivity

Copy link

Marking this PR as stale due to inactivity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new feature request For possible new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants