ref(feedback): Align tooltip defaults with page-frame

Use page-frame as the single default tooltip path for FeedbackButton and keep non-page-frame behavior unchanged unless callers provide their own tooltip.

This matches the agreed scope and removes the extra icon-only branching from the shared component.

Refs DE-1104
Co-Authored-By: OpenAI Codex <noreply@openai.com>

Author: priscilawebdev

static/app/components/feedbackButton/feedbackButton.tsx

@@ -63,15 +63,13 @@ export function FeedbackButton({feedbackOptions, ...buttonProps}: Props) {
   }
 
   const defaultLabel = t('Give Feedback');
-  const isIconOnly = buttonProps.children === null;
-  const children = isIconOnly ? null : (buttonProps.children ?? defaultLabel);
-  const ariaLabel = buttonProps['aria-label'] ?? (isIconOnly ? defaultLabel : undefined);
-  const tooltipTitle =
+  const children = buttonProps.children ?? (hasPageFrameFeature ? null : defaultLabel);
+  const label =
     buttonProps['aria-label'] ?? (typeof children === 'string' ? children : defaultLabel);
   const tooltipProps =
-    buttonProps.tooltipProps?.title || (!hasPageFrameFeature && !isIconOnly)
+    buttonProps.tooltipProps?.title || !hasPageFrameFeature
       ? buttonProps.tooltipProps
-      : {...buttonProps.tooltipProps, title: tooltipTitle};
+      : {...buttonProps.tooltipProps, title: label};
 
   return (
     <Button
@@ -80,7 +78,7 @@ export function FeedbackButton({feedbackOptions, ...buttonProps}: Props) {
       icon={<IconMegaphone />}
       {...buttonProps}
       tooltipProps={tooltipProps}
-      aria-label={ariaLabel}
+      aria-label={children === null ? label : buttonProps['aria-label']}
       onClick={e => {
         openForm?.(feedbackOptions);
         buttonProps.onClick?.(e);