diff --git a/packages/feedback/src/core/sendFeedback.ts b/packages/feedback/src/core/sendFeedback.ts index 3848eb9cbc33..44922d785f2a 100644 --- a/packages/feedback/src/core/sendFeedback.ts +++ b/packages/feedback/src/core/sendFeedback.ts @@ -48,17 +48,19 @@ export const sendFeedback: SendFeedback = ( if ( response && typeof response.statusCode === 'number' && - (response.statusCode < 200 || response.statusCode >= 300) + response.statusCode >= 200 && + response.statusCode < 300 ) { - if (response.statusCode === 0) { - return reject( - 'Unable to send Feedback. This is because of network issues, or because you are using an ad-blocker.', - ); - } - return reject('Unable to send Feedback. Invalid response from server.'); + resolve(eventId); } - resolve(eventId); + if (response && typeof response.statusCode === 'number' && response.statusCode === 0) { + return reject( + 'Unable to send Feedback. This is because of network issues, or because you are using an ad-blocker.', + ); + } + + return reject('Unable to send Feedback. Invalid response from server.'); }); }); }; diff --git a/packages/feedback/src/modal/components/Dialog.css.ts b/packages/feedback/src/modal/components/Dialog.css.ts index 76ac68f0d6d1..a9d1f4331606 100644 --- a/packages/feedback/src/modal/components/Dialog.css.ts +++ b/packages/feedback/src/modal/components/Dialog.css.ts @@ -100,7 +100,8 @@ const FORM = ` } .form__right { - min-width: var(--form-width, 272px); + flex: 0 0 var(--form-width, 272px); + width: var(--form-width, 272px); display: flex; overflow: auto; flex-direction: column; diff --git a/packages/feedback/src/modal/components/Form.tsx b/packages/feedback/src/modal/components/Form.tsx index 788512cb192b..71c597e62c9d 100644 --- a/packages/feedback/src/modal/components/Form.tsx +++ b/packages/feedback/src/modal/components/Form.tsx @@ -128,7 +128,7 @@ export function Form({ onSubmitSuccess(data); } catch (error) { DEBUG_BUILD && logger.error(error); - setError('There was a problem submitting feedback, please wait and try again.'); + setError(error as string); onSubmitError(error as Error); } } catch { @@ -228,7 +228,11 @@ function LabelText({ label, isRequired, isRequiredLabel, -}: { label: string; isRequired: boolean; isRequiredLabel: string }): VNode { +}: { + label: string; + isRequired: boolean; + isRequiredLabel: string; +}): VNode { return ( {label}