Skip to content

Keep feedback launcher visible on mobile - #121

Merged
sarthakagrawal927 merged 1 commit into
mainfrom
agent/swe-feedback-mobile-visibility-20261004
Oct 4, 2026
Merged

sarthakagrawal927 merged 1 commit into
mainfrom
agent/swe-feedback-mobile-visibility-20261004

Conversation

@sarthakagrawal927

Copy link
Copy Markdown
Member

Cause

A global @media (max-width: 1024px) rule in src/index.css set [data-saasmaker-widget] to display: none !important, hiding the feedback launcher on phones and tablets.

Change

Removed that hide rule. The existing widget keeps its normal bottom-right placement; the compact navigation remains in the header, and no positioning override was needed.

Verification

  • git diff --check passed.
  • Local Biome and Vite build could not run because this managed macOS workspace lacks the optional Biome and Rolldown native bindings. Linux CI and rendered review are pending.
  • Existing feedback route guards and widget implementation are unchanged.

@sarthakagrawal927
sarthakagrawal927 merged commit 54d52e0 into main Oct 4, 2026
1 check passed
@sarthakagrawal927

Copy link
Copy Markdown
Member Author

Released and verified at exact main 54d52e0b89793e195f2a055101ed6080e3a9c447.

  • PR and exact-main CI passed; guarded deployment run 37178178134 succeeded with migrations skipped.
  • Provider production deployment: 8ba66749-42c6-4694-9c55-c01733812415, source 54d52e0.
  • Real owner Chrome: feedback button and dialog visible and operable at 390, 768 and 1440px. Fresh settled captures saved locally; no form submitted. Temporary viewport reset and QA tab closed.
  • This removes the mobile CSS suppression while preserving the existing focused-route source guard. Valid share route rendered acceptance remains unverified; an invalid share URL is not evidence.
  • No dependencies, secrets, migrations or production-config changes in this patch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant