Repository navigation
Add emoji picker with Autocomplete grid #83538
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
Open
adamsilverstein
wants to merge
445
commits into
trunk
Choose a base branch
from
try/notes-emoji-picker-autocomplete-grid
base: trunk
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
445 commits
Select commit
Hold shift + click to select a range
ab18eaa
Notes: Add E2E tests pinning the popover-portal and selection-persist…
adamsilverstein 8576647
Notes: Wrap Frimousse popover tightly to its picker
adamsilverstein 6021c5a
Notes: Size Frimousse picker to its 8-column grid, not an arbitrary s…
adamsilverstein 459d7c6
Notes: Allow re-adding a reaction after removing it on the same note
adamsilverstein 45be10c
Notes: Drop empty wordpress-7.0/block-comments.php compat shim
adamsilverstein fe23e2a
Notes: Force-delete reactions and refetch only the toggled note's record
adamsilverstein ac175cc
Notes: Replace Frimousse picker with native SearchControl + Composite…
adamsilverstein f0038b9
Notes: Move full emoji picker to a follow-up PR
adamsilverstein e02e83e
Notes: Drop hex-key storage helpers and orphan picker CSS
adamsilverstein 0734bf1
Notes: Consolidate reaction failure-case tests with a data provider
adamsilverstein 26a6739
Notes: Switch curated reaction picker from vertical to horizontal
adamsilverstein 49cc701
Update backport-changelog/7.0/10930.md
adamsilverstein bafb480
Update packages/editor/src/components/collab-sidebar/style.scss
adamsilverstein 0a3e814
Notes: Standardize add-reaction button to 24px height
adamsilverstein e25ad38
Notes: Disable add-reaction button on resolved threads
adamsilverstein e89d3f5
Notes: Switch emoji slug lookups to a memoized Map
adamsilverstein 38229af
Notes: Inline smiley icon, drop from public @wordpress/icons
adamsilverstein 7b497a6
Notes: Harden note and reaction creation permissions
adamsilverstein a21001b
Notes: Stop overriding note children link with reactions
adamsilverstein 8182634
Notes: Drop OPTIONS emoji fetch in favor of curated client list
adamsilverstein 405c556
Notes: Extend the core comment controller instead of the removed 6.9 …
adamsilverstein e85f85c
Notes: Use user-facing role selectors for reaction-button e2e assertions
adamsilverstein 95f250d
Build: Drop the stray 'Step 7.5' comment from build.mjs
adamsilverstein 773292d
Notes: Restrict reaction read access to author and editors
adamsilverstein 5890c09
Notes: Exclude reactions from the comment-count query per internal type
adamsilverstein c52939b
Notes: Canonicalize stored reaction slug and reject invalid codepoints
adamsilverstein 0c02b3f
Notes: Invalidate reaction tooltip cache when adding via the picker
adamsilverstein 95d3586
Notes: Harden reaction tests against leaked filters and snackbar timing
adamsilverstein 8809f28
Notes: Converge concurrent duplicate reactions to a single row
adamsilverstein 95535a2
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 531bcf4
Notes: Re-select the note thread before removing a reaction in e2e
adamsilverstein 735dd6a
Merge branch 'trunk' into add-notes-emoji-reactions-try-addditional-c…
adamsilverstein 04ff952
Notes: Add unit tests for reaction display and emoji picker
adamsilverstein 0ecba04
Notes: Add test that notes and reactions stay out of comment counts
adamsilverstein 0b58a36
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 0c65a93
Add translator context to reaction emoji labels
adamsilverstein e3b3a5d
Merge branch 'trunk' into add-notes-emoji-reactions-try-addditional-c…
adamsilverstein b7f88e6
Merge branch 'trunk' into add-notes-emoji-reactions-try-addditional-c…
adamsilverstein 786eebb
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein 033b1cf
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein d05c91d
Notes reactions: use standard Button styles for triggers and pills
adamsilverstein 3fc3c8a
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 8cf346a
Notes full picker: address design review feedback
adamsilverstein 5e19a13
Notes reactions: use accent tint for the reacted pill state
adamsilverstein ec5f642
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein e7bdbc6
Add Storybook story for the Notes emoji picker
adamsilverstein 9a682b5
Add Storybook story for the full Notes reaction row
adamsilverstein ed4c39b
Consolidate reaction triggers into a single smiley button
adamsilverstein e41a6eb
Add a skin tone picker to the full emoji picker
adamsilverstein b4a2ee4
Add e2e coverage for the emoji skin tone preference
adamsilverstein 3dd4f26
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein 50aacc5
Add a Frequently used section to the full emoji picker
adamsilverstein e984905
Add e2e coverage for the Frequently used emoji section
adamsilverstein 496d549
Notes: Author the emoji picker modules as TypeScript
adamsilverstein 6179be8
Remove reaction added/removed snackbars
adamsilverstein 3af9768
Convert new reaction modules to TypeScript
adamsilverstein 05c6c6b
Wire the reaction emoji filter to the picker and cover it with e2e
adamsilverstein 36d7796
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein a8779a7
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein b7dfa8c
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein 379bdff
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein ff10c2f
Notes: Fix build failures from trunk merge
adamsilverstein 0d9df50
Notes: Drop stale 'Reaction added.' snackbar assertions from e2e
adamsilverstein 6bc2fec
Merge trunk into add-notes-emoji-reactions-try-addditional-comment-type
adamsilverstein 70cff94
Merge updated base (trunk-current #76767) into full-picker
adamsilverstein 76ccc26
Notes: Decode emoji code points without mbstring
adamsilverstein a06fd32
Notes: Normalize full-picker picks against the filtered emoji list
adamsilverstein ccfff03
Notes: Announce emoji search result counts
adamsilverstein f92a981
Notes: Focus the selected skin tone when the flyout opens
adamsilverstein ac9d380
Notes: Expose picker categories to AT and flatten search results
adamsilverstein e883e28
Notes: Match the emoji picker popover to standard popover chrome
adamsilverstein d9a475b
Notes: Add editor CHANGELOG entry for the full emoji picker
adamsilverstein a52e9f9
Merge trunk into add-notes-emoji-reactions-try-addditional-comment-type
adamsilverstein d08d3e5
Merge branch 'add-notes-emoji-reactions-try-addditional-comment-type'…
adamsilverstein 6ede323
Notes: Open the full emoji picker directly from the add-reaction button
adamsilverstein 5643a47
Notes: Fix emoji grid focus ring clipping and phantom active highlight
adamsilverstein d776cff
Notes: Drop the redundant 'emoji' suffix from reaction tooltips
adamsilverstein 1d04ff3
Notes: Give reaction pills more breathing room
adamsilverstein f38bc94
Notes: Expose the skin-tone listbox's horizontal orientation
adamsilverstein 755bc44
Notes: Derive the reactor tooltip from names plus the current label
adamsilverstein d6b6d55
Notes: Name the reaction picker popups and advertise them on triggers
adamsilverstein 777f0b8
Notes: Extract the add-reaction picker and survive full-picker failures
adamsilverstein 25313a8
Notes: Configure the emoji picker through editor settings, not globals
adamsilverstein a1e7d01
Notes: Derive picker geometry from WPDS tokens
adamsilverstein 8d973ba
Notes: Assert Escape focus return and nested-overlay dismissal in e2e
adamsilverstein 57d5348
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein e2fa31a
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 8f87eca
Notes: Fix reaction picker layout and arrow-key navigation
adamsilverstein a5b27f2
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein 7d66848
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein db8ac3a
Merge branch 'add-notes-emoji-reactions-try-addditional-comment-type'…
adamsilverstein 5209f0d
Update the picker keynav test and drop stale dependency-group comments
adamsilverstein bb8206e
Drop stale dependency-group comments and hoist a stray import
adamsilverstein 7a10cd5
Merge branch 'add-notes-emoji-reactions-try-addditional-comment-type'…
adamsilverstein 73e59d3
Merge trunk into add-notes-emoji-reactions-try-addditional-comment-type
adamsilverstein 011b99b
Adapt reaction tests to trunk's typechecked dev projects
adamsilverstein f1f9816
Editor: reference primitives from the build tsconfig project
adamsilverstein 9e3aff6
Restrict reaction writes to create and delete
adamsilverstein fae6d45
Remove a note's reactions when the note goes
adamsilverstein 9ef52cf
Present the reaction picker as a group of buttons
adamsilverstein c1ddeb7
Lock reactions on a resolved note thread
adamsilverstein 193fed6
Page through reactions before naming reactors
adamsilverstein fcb732e
Keep reaction state consistent when the refetch fails
adamsilverstein 1e051ca
Fix the concurrent-cleanup regression test setup
adamsilverstein d8018c7
Drive the resolved-reaction test through the sidebar
adamsilverstein ca95c99
Keep reaction hex keys in one normal form
adamsilverstein 2905c6e
Fix the emoji picker's loading, announcement and tooltip edges
adamsilverstein 1d1681b
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein 4edeefd
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 084bf12
Align reaction tests with the merged curated picker
adamsilverstein 1bd63da
Notes reactions: quiet the reaction pills
adamsilverstein 669b089
Convert the emoji picker's new test and story files to TypeScript
adamsilverstein fddf20e
Notes reactions: give the pills room and quiet the trigger at rest
adamsilverstein 19164b4
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 53380fe
Add backport changelog entry for PR #78176
adamsilverstein 649b766
Accept hex code-point reaction slugs from the full picker
adamsilverstein 821d62f
Notes reactions: build the pills from Design System buttons
adamsilverstein f3a6248
Match the reaction tooltip by its text
adamsilverstein 366e53f
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 73d812b
Drop 'emoji' from the reaction tooltip assertion
adamsilverstein 5c6367a
Drop 'emoji' from the reaction tooltip assertion
adamsilverstein 8daff84
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein 27ffc8c
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 85042b8
Align reaction row height and outline unpressed pills
adamsilverstein 9713a89
Merge commit '85042b89cbd' into add-notes-emoji-reactions-full-picker
adamsilverstein 7e3a89c
Keep the add-reaction trigger at button height when pills wrap
adamsilverstein 5294abb
Merge commit '7e3a89c33df' into add-notes-emoji-reactions-full-picker
adamsilverstein 12b68c2
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein 6208740
Rename reaction tests to the jsdom test naming convention
adamsilverstein a1ccbcc
Merge branch 'add-notes-emoji-reactions-try-addditional-comment-type'…
adamsilverstein 7880ae3
Rename emoji picker tests to the jsdom test naming convention
adamsilverstein 6d0c29b
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein 669b874
Merge branch 'add-notes-emoji-reactions-try-addditional-comment-type'…
adamsilverstein 5b98514
Move note reaction compat code to the WordPress 7.2 directory
adamsilverstein 3c6e363
Merge branch 'add-notes-emoji-reactions-try-addditional-comment-type'…
adamsilverstein 11a5127
Move the emojibase data compat file to the WordPress 7.2 directory
adamsilverstein 8c17d82
Tighten inline documentation across the emoji picker
adamsilverstein 4b59106
Add editor changelog entry for note emoji reactions
adamsilverstein 1aed735
Merge commit '4b591065801' into add-notes-emoji-reactions-full-picker
adamsilverstein 06558d6
Move the emoji picker changelog entry to Unreleased
adamsilverstein 4e66600
Emoji picker: use Unicode's own category names for the group headings
adamsilverstein 5787b53
Merge trunk into add-notes-emoji-reactions-full-picker
adamsilverstein 62a9f07
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein 44f5e24
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 10f51f9
Tests: Run the reaction tests under Vitest
adamsilverstein 0467594
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 5bca0cf
Tests: Run the emoji picker tests under Vitest
adamsilverstein 1948472
Tests: Type the api-fetch module mock
adamsilverstein ab14021
Tests: Type the module mocks and mock matchMedia for the skin tone fl…
adamsilverstein f2fb3f5
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 0ef5a47
Editor: Declare the Vitest test env types for the dev project
adamsilverstein b0cfa57
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein 3add4b4
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 05b2615
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein 5dd9ae6
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein ef0e27f
Reformat ternary continuations for Prettier 3.9.6
adamsilverstein 75625d0
Notes: reveal the add-reaction trigger on hover instead of reserving …
adamsilverstein e8c4f13
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein 7f143b6
Notes: Use the reaction icon from @wordpress/icons
adamsilverstein 25e7637
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 59ad5e8
Notes: Use the reaction icon for the add reaction trigger
adamsilverstein 4d22b2d
Notes: Reveal the add-reaction trigger on hover
adamsilverstein 1c7479e
Merge #76767 into #78176 now that it carries the hover trigger
adamsilverstein 98ab3b0
Notes: Prefetch reaction summaries from the real result set
adamsilverstein 2622bee
Notes: Keep a note's edit-context fields when refreshing reactions
adamsilverstein e0febbf
Merge branch 'add-notes-emoji-reactions-try-addditional-comment-type'…
adamsilverstein 16369f1
Merge remote-tracking branch 'origin/trunk' into add-notes-emoji-reac…
adamsilverstein e789531
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein 133b993
Notes: Keep the add-reaction trigger visible once a note has reactions
adamsilverstein c9c7f1c
Merge remote-tracking branch 'origin/add-notes-emoji-reactions-try-ad…
adamsilverstein fd77835
Notes: Build the full emoji picker on the Autocomplete grid
adamsilverstein 01050f1
Notes: Cover the Autocomplete grid keyboard model in tests
adamsilverstein b86ad40
Notes: Drop emoji picker styles the Autocomplete item already covers
adamsilverstein e7430b5
Notes: Keep the stuck emoji heading clear of the search focus ring
adamsilverstein 53b4cd5
Notes: Use the Design System highlight for emoji picker cells
adamsilverstein 946347d
Notes: Skip the scrollable-region axe rule on the emoji picker story
adamsilverstein ef80844
Notes: Compose the emoji picker on the documented Autocomplete async …
adamsilverstein 924493b
Notes: Match the emoji picker spacing to the updated design
adamsilverstein 897ca6e
Notes: Keep the emoji picker popover width stable
adamsilverstein fd9257d
Merge remote-tracking branch 'origin/trunk' into try/notes-emoji-pick…
adamsilverstein f9f00d2
Wrap emoji picker cells in Autocomplete.ItemLabel
adamsilverstein a72dead
Move backport changelog entry for core PR 10930 to 7.2
adamsilverstein 7b36a18
List PR 83538 in the backport changelog entry for core PR 10930
adamsilverstein 79156fe
Notes: Exclude reactions from comment feeds
adamsilverstein a116471
Notes: Keep reactor names when the last reaction page is full
adamsilverstein 86ce44e
Notes: Copy emoji data in dev builds and skip the URL without it
adamsilverstein 90417be
Notes: Space emoji search results clear of the search field
adamsilverstein 247055d
Notes: Pin reactions to the current user
adamsilverstein 8e38936
Notes: Fix reaction cascades for trashed notes
adamsilverstein f9d74a1
Notes: Ignore reaction summary refreshes a newer toggle overtook
adamsilverstein 893af05
Tests: Mock Openverse in the image drag-and-drop e2e test
adamsilverstein df3fe34
Tests: Bump undici in the AI development test lockfile
adamsilverstein 066ac30
Notes: Reset frequent emoji preference between picker tests
adamsilverstein 97bd41a
Notes: Accept content.raw when validating a reaction slug
adamsilverstein e0fa127
Notes: Keep focus in the reaction popover when the picker falls back
adamsilverstein 1714c04
Notes: Pad the end of the emoji grid to match its sides
adamsilverstein e22d5e1
Merge remote-tracking branch 'origin/trunk' into try/notes-emoji-pick…
adamsilverstein 5068548
Notes: Keep an emptied reaction emoji list empty
adamsilverstein 1966b99
Notes: Count a deduplicated reaction add once
adamsilverstein 6dce7bc
Notes: Reject reactions on trashed or spam notes
adamsilverstein 0c99115
Notes: Only clear the reaction trash flag on reactions
adamsilverstein d8f77d6
Notes: Fall back to default internal comment types when filtered empty
adamsilverstein 9fe5628
Notes: Let sites limit which emoji reactions accept
adamsilverstein 828f3a2
Notes: Apply the emoji rules in the reaction picker
adamsilverstein 602ad3b
Notes: Merge the reaction emoji filters into one
adamsilverstein ba5b98c
Merge remote-tracking branch 'origin/trunk' into try/notes-emoji-pick…
adamsilverstein 00116f1
Merge remote-tracking branch 'origin/trunk' into try/notes-emoji-pick…
adamsilverstein 4597ebb
Merge remote-tracking branch 'origin/trunk' into try/notes-emoji-pick…
adamsilverstein 557d1e9
Notes: Announce the emoji grid once after a pending load
adamsilverstein d602a9a
Notes: Drop the inline padding on emoji category headings
adamsilverstein 769b972
Notes: Remove emoji picker stories
adamsilverstein 0b055e4
Notes: Remove emoji reaction unit tests
adamsilverstein 4b76e27
Notes: Prune emoji reaction e2e tests
adamsilverstein 80507d4
Notes: Prune emoji reaction PHP tests
adamsilverstein 17cb563
Merge remote-tracking branch 'origin/trunk' into try/notes-emoji-pick…
adamsilverstein c60df22
Notes: Drive the emoji picker entirely through Autocomplete
adamsilverstein f053bf0
Notes: Group reactions under a subdirectory with a useReaction hook
adamsilverstein beea594
UI: Add Autocomplete.Trigger
adamsilverstein 0b79c1c
Notes: Open the emoji picker from Autocomplete.Trigger
adamsilverstein 4596159
Notes: Build the skin tone picker with Menu
adamsilverstein 1e03b55
Notes: Pass noteReactionEmojiRules through to the block editor
adamsilverstein b72a532
Notes: Cap Frequently used at five full grid rows
adamsilverstein efa5753
Notes: Bring reaction comments in line with the current picker
adamsilverstein b84d968
Notes: Remove the emoji reaction customization filters
adamsilverstein a0209ea
Notes: Store every emoji reaction as its hex key
adamsilverstein ee6eea5
Notes: Show an error instead of a fallback when emojis fail to load
adamsilverstein 1c1c94a
Storybook: Add Autocomplete.Trigger to the components manifest
adamsilverstein 50587a1
Notes: Recover the emoji picker from a failed or malformed dataset load
adamsilverstein 250fb4e
Notes: Fetch a note's reactor names once for all its pills
adamsilverstein 29d1d5e
Merge branch 'trunk' into try/notes-emoji-picker-autocomplete-grid
adamsilverstein 6b56b7b
Notes: Lowercase the curated reaction emoji names
adamsilverstein 1405e84
Merge trunk into try/notes-emoji-picker-autocomplete-grid
adamsilverstein 1de738f
Backport changelog: List only this PR for wpd #10930
adamsilverstein 3c9014a
Notes: Drop the reaction restore cascade on untrash
adamsilverstein 5c02241
Notes: Collapse reaction_summary to count and current_user_reaction
adamsilverstein d9600a9
Notes: Use the ui Tooltip on the skin tone toggle
adamsilverstein 61e250d
Notes: Keep emoji glyphs in color under forced colors
adamsilverstein b29a486
Notes: Keep the emoji grid scroller out of the Tab order
adamsilverstein c009f9a
Merge remote-tracking branch 'origin/trunk' into try/notes-emoji-pick…
adamsilverstein a7d2fba
Notes: Skip the reply composer in the reaction hover e2e test
adamsilverstein 97843a8
Notes: Enforce reaction ownership and resolved threads on the server
adamsilverstein 912d09c
Notes: Let forced colors repaint reaction emoji
adamsilverstein fc57daa
Notes: Stop the thread root walk on a cyclic parent chain
adamsilverstein 80ac9ba
Notes: Guard reaction toggles against stale and repeated clicks
adamsilverstein aac6a41
Merge remote-tracking branch 'origin/trunk' into try/notes-emoji-pick…
adamsilverstein a251c4b
Notes: Inline the internal comment types instead of a helper
adamsilverstein f6a32e2
Notes: Localize the reaction pill count
adamsilverstein 7e95aeb
Notes: Mark own reactions in the picker and hide it on resolved threads
adamsilverstein 74c7340
Notes: Fix emoji picker type errors
adamsilverstein a217fd2
Merge remote-tracking branch 'origin/trunk' into try/notes-emoji-pick…
adamsilverstein File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| https://github.com/WordPress/wordpress-develop/pull/10930 | ||
|
|
||
| * https://github.com/WordPress/gutenberg/pull/83538 | ||
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.