Open UnderKoen opened 4 weeks ago
Name | Link |
---|---|
Latest commit | cab4e2ab1c963c8dada82d6707d28cd1fc01307b |
Latest deploy log | https://app.netlify.com/sites/actualbudget/deploys/67292323a6e49300081c6b9f |
Deploy Preview | https://deploy-preview-3703.demo.actualbudget.org |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Hey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle.
As this PR is updated, I'll keep you updated on how the bundle size is impacted.
Total
Files count | Total bundle size | % Changed |
---|---|---|
10 | 5.34 MB → 5.34 MB (+805 B) | +0.01% |
Hey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle.
As this PR is updated, I'll keep you updated on how the bundle size is impacted.
Total
Files count | Total bundle size | % Changed |
---|---|---|
1 | 1.27 MB | 0% |
Changeset
No files were changed
[!WARNING]
Rate limit exceeded
@UnderKoen has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 1 minutes and 30 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
Reviewing files that changed from the base of the PR and between 87835c7c6bdf10a5ab97f5f8e78b1bfbca6ae309 and cab4e2ab1c963c8dada82d6707d28cd1fc01307b.
The changes in the pull request focus on the MonthPicker
component located in MonthPicker.tsx
. The useTranslation
hook has been imported from react-i18next
, along with the SvgCalendar
component from ../../icons/v2
. A new "Now" button has been introduced, which, when clicked, invokes the onSelect
function with the current month as its argument. This button is styled with specific attributes and is positioned absolutely within the component's layout.
The overall structure of the component remains unchanged, maintaining the existing hover and selection logic. The t
function from useTranslation
is used to set the button's title to "Now". The method signature for onSelect
in MonthPickerProps
has been updated to reflect the context of usage with the addition of the "Now" button, although the signature itself has not changed. The component continues to display months responsively, allowing users to select a month with the current month highlighted.
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?
test again
moved back to WIP like it was before
Can you make this not take up extra vertical space? Maybe put the button in line with the picker somewhere instead of over it?
I agree with @youngcw about making the now button in line with the picker.
Another suggestion is to put the now button under the picker so that the user knows where to look, the Now is kinda sitting with a lot of whitespace around it to me.
And thank you for getting this started!
:wave: Hi! It looks like this PR has not had any changes for a week now. Would you like someone to review this PR? If so - please remove the "[WIP]" prefix from the PR title. That will let the community know that this PR is open for a review.
/update-vrt
/update-vrt
/update-vrt
/update-vrt
Would it work to use the calendar icon we have, or would that be confusing?
In the dark modes something about my monitor/eyes makes the now button look like its glowing almost like the color changing buttons you see on discord.
It definitely looks cleaner! But I would expect a date picker widget when I press it with this icon. What do you think?
Isn't it common to have a calendar icon as a "today" button? I feel like that gets used plenty
/update-vrt
Instead of "Now" could the hover text say "Today"? And I think it would look nicer if the button stayed with the month picker instead of being pinned to the far left header.
It looks great! Thanks for all the good work.
One bug I found is that the calendar button when multiple months are displayed is too far to the left; it's not flush with the left side of the month.