-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Add resize service #577
Add resize service #577
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the implementation!! I think that specific utility services are better than one big WindowService
.
- Need to add a
mod resize;
statement insidesrc/services/mod.rs
- Please re-export the
ResizeService
just like the other services
…dd doc comments, add window dimensions when the callback is called
Sorry about that, I've had this service lying around for the greater part of a year and I just copied+pasted them into Yew without doing a proper check first. Coding before coffee leads to these mistakes 😄. |
Haha no worries, thanks for addressing the feedback with lightning speed 👍 |
bors r+ |
577: Add resize service r=jstarry a=hgzimmerman Adds a service to handle resize events. Implementation as loosely dictated by #541. If you want to make this a full-blown window service instead of just focusing on resizes, let me know the other events this service should handle and I'll try to add them. Co-authored-by: Henry Zimmerman <zimhen7@gmail.com>
Build succeeded
|
Adds a service to handle resize events.
Implementation as loosely dictated by #541.
If you want to make this a full-blown window service instead of just focusing on resizes, let me know the other events this service should handle and I'll try to add them.