Added a button to the Story Budget date change option #509
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
|
Thanks for adding this! I made a minor change to relocate the Today button beside the date field for better context |
cojennin
reviewed
Dec 11, 2019
mjangda
reviewed
Dec 13, 2019
mjangda
reviewed
Dec 13, 2019
Add a "Today" button to the Story Budget page that allows users to quickly jump to the current date without needing to use the date picker. The button: - Uses current_time() for timezone-accurate dates - Preserves the user's existing "number of days" setting - Uses a <button> element to avoid CSS conflicts with hidden inputs Also improves the redirect by using wp_safe_redirect() with exit instead of wp_redirect() with wp_die(). Fixes Automattic#176 🤖 Generated with [Claude Code](https://claude.ai/claude-code) Co-Authored-By: Radu Constantin <raduconst@users.noreply.github.com> Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
GaryJones
force-pushed
the
story-budget-today-button
branch
from
e30020b to
fda6229
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I added the button to let the user go faster to the current day instead of having to use the
calendar. I tried to keep the same behavior of the Today button from Edit Flow → Calendar → Customizable calendar, where you can go to the current date with just a press of a button.
P.S. If this PR is accepted, the pot file needs to be generated again because there's a
orword between theChangeandTodaybuttons.This is made to fix #176