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

extend event class & add event tile height #42

Merged
merged 2 commits into from
Oct 3, 2022
Merged

extend event class & add event tile height #42

merged 2 commits into from
Oct 3, 2022

Conversation

jnelle
Copy link
Contributor

@jnelle jnelle commented Sep 16, 2022

Hi,

first of all, thanks for maintaining this package!

I faced an issue with the event tile (check screenshot), that's why I added the height attribute to the widget.
Also in my case it was important save additional informations in an event object, that's why I added an attribute to the event class.

image

Signed-off-by: jnelle <jimmy@nelle.sh>
* also formaatted the file
* I had to add the height, because I faced a renderoverflow

Signed-off-by: jnelle <jimmy@nelle.sh>
lib/neat_and_clean_calendar_event.dart Outdated Show resolved Hide resolved
@rwbr
Copy link
Owner

rwbr commented Sep 18, 2022

Introducing the height property for the event tiles is a good idea.

@jnelle
Copy link
Contributor Author

jnelle commented Sep 23, 2022

Hi, I changed it and gonna open a new one when this pr gets merged

@jnelle
Copy link
Contributor Author

jnelle commented Sep 28, 2022

ping

@jnelle
Copy link
Contributor Author

jnelle commented Oct 3, 2022

Hi, is it possible to merge this now?

@rwbr
Copy link
Owner

rwbr commented Oct 3, 2022

Sure, sorry for the delay

@rwbr rwbr merged commit 9ea7b8f into rwbr:main Oct 3, 2022
rwbr pushed a commit that referenced this pull request Oct 3, 2022
* Includes PR #42 extend event class & add event tile height
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants