Closed hcavarsan closed 2 months ago
[!WARNING]
Rate limit exceeded
@hcavarsan has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 32 minutes and 37 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between b41ee2aa9b0963bb01b64e5730bb47b2018345ca and aa3b75e85ec088eb29eeaf0dcbd24b762745e997.
The recent updates to the kftray-tauri
crate involve significant enhancements and refactoring. Key changes include the addition of a new is_plugin_moving
field in AppState
, modifications to window position handling by changing data types from f64
to i32
, and the introduction of new functions to manage window size, position, and visibility. These changes aim to improve the robustness and functionality of the application's window management system.
Files | Change Summary |
---|---|
crates/kftray-tauri/src/main.rs |
Added is_plugin_moving field to AppState and updated its declaration. Imported AtomicBool . |
crates/kftray-tauri/src/models/window.rs |
Changed WindowPosition fields x and y from f64 to i32 . Added Debug trait to the struct. |
crates/kftray-tauri/src/tray.rs |
Added functions adjust_window_size_and_position , set_default_position . Handled WindowEvent::ScaleFactorChanged . |
crates/kftray-tauri/src/window.rs |
Refactored functions and types for window position management. Added new functions for error handling and state reset. |
sequenceDiagram
participant User
participant App
participant Window
User->>App: Launch Application
App->>Window: Initialize with default position
Window-->>App: Position set to default
App->>Window: Toggle visibility
Window-->>App: Visibility toggled
App->>Window: Save window position
Window-->>App: Position saved
App->>Window: Adjust size and position
Window-->>App: Size and position adjusted
Note over App, Window: Updates `is_plugin_moving` state
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
fix: handle better window position size with multiple monitors and scalefactorchanged