Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
Significance: patch
Type: changed

Feedback: tell readers where their feedback went and that they can send more, instead of a one-line thank you.
Original file line number Diff line number Diff line change
Expand Up @@ -86,8 +86,17 @@ export function FeedbackModal( { onClose }: FeedbackModalProps ) {
<>
<Dialog.Content>
<Notice.Root intent="success">
<Notice.Title>
{ __(
'Thanks, your feedback has gone to the team.',
'jetpack-premium-analytics-pkg'
) }
</Notice.Title>
<Notice.Description>
{ __( 'Thank you. This helps.', 'jetpack-premium-analytics-pkg' ) }
{ __(
"It'll help us decide what to fix before the new Traffic tab replaces the old one. You can send more any time from the same menu.",
'jetpack-premium-analytics-pkg'
) }
</Notice.Description>
</Notice.Root>
</Dialog.Content>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -149,8 +149,12 @@ describe( 'DashboardOptionsMenu', () => {

// Scoped to the dialog: `Notice` also mirrors the text into the a11y-speak live
// region on `body`, so an unscoped query matches twice.
const dialog = within( screen.getByRole( 'dialog' ) );
expect( dialog.getByText( 'Thanks, your feedback has gone to the team.' ) ).toBeInTheDocument();
expect(
within( screen.getByRole( 'dialog' ) ).getByText( 'Thank you. This helps.' )
dialog.getByText(
"It'll help us decide what to fix before the new Traffic tab replaces the old one. You can send more any time from the same menu."
)
).toBeInTheDocument();
expect( screen.queryByRole( 'radiogroup' ) ).not.toBeInTheDocument();

Expand Down Expand Up @@ -296,8 +300,12 @@ describe( 'the Happiness copy of the feedback', () => {
await user.type( screen.getByRole( 'textbox' ), 'Charts load slowly' );
await user.click( screen.getByRole( 'button', { name: 'Send feedback' } ) );

const dialog = within( screen.getByRole( 'dialog' ) );
expect( dialog.getByText( 'Thanks, your feedback has gone to the team.' ) ).toBeInTheDocument();
expect(
within( screen.getByRole( 'dialog' ) ).getByText( 'Thank you. This helps.' )
dialog.getByText(
"It'll help us decide what to fix before the new Traffic tab replaces the old one. You can send more any time from the same menu."
)
).toBeInTheDocument();
} );
} );
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
Significance: patch
Type: enhancement

Premium Analytics: tell readers where their feedback went and that they can send more, instead of a one-line thank you.
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
Significance: patch
Type: changed

Feedback: tell readers where their feedback went and that they can send more, instead of a one-line thank you.
Loading