Skip to content

Conversation

@C0mberry
Copy link
Contributor

@C0mberry C0mberry commented Dec 23, 2025

Description

that fix allows you to interact with input on a mobile phone and as a result insert text

Type of change

  • Bug fix
  • Documentation update

Screenshots

Screenshot 2025-12-18 at 17 48 19

video from mobile

Testing

  1. open http://localhost:3000/ (main page) on mobile
  2. try to paste anything clicking by any place inside select

Related Issues

HOO-216

Checklist

  • My code follows the project's style guidelines
  • I have added tests that prove my fix/feature works
  • All tests pass locally and in CI
  • CI/CD checks pass
  • I have included screenshots for protocol screens (if applicable)

* fix input width

* move styles for conrtol back

* chore: add section into docs about running app locally with https enabled

* chore: add story to test out SearchBar pointer-events handling

* nit: codestyle

* chore: improve story to check for interactiveness criteria

---------

Co-authored-by: Sergo <rogaldh@radsh.red>
@vercel
Copy link

vercel bot commented Dec 23, 2025

@C0mberry is attempting to deploy a commit to the Solana Foundation Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

Looks good to me! 👍

Reviewed everything up to 2800cc2 in 2 minutes and 5 seconds. Click for details.
  • Reviewed 110 lines of code in 3 files
  • Skipped 1 files when reviewing.
  • Skipped posting 4 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. app/components/SearchBar.tsx:160
  • Draft comment:
    Add a comment explaining why the gridTemplateColumns style ('0 minmax(min-content, 1fr)') is applied to the input. This helps future maintainers understand it's a workaround for the react-select issue (github.com/JedWatson/react-select/issues/3857) and ensures proper mobile touch behavior.
  • Reason this comment was not posted:
    Decided after close inspection that this draft comment was likely wrong and/or not actionable: usefulness confidence = 20% vs. threshold = 50% The existing comment on line 163 already mentions the react-select issue #3857 as a workaround. The comment is positioned right after the input style block where gridTemplateColumns was added. While the existing comment specifically mentions it's for the placeholder style, it's reasonable to infer that the nearby input style changes are related to the same issue. The suggested comment would add clarity, but it's not strictly necessary since there's already a reference to the issue. The comment is asking for documentation/explanation rather than pointing out a code issue. According to the rules, comments should not be purely informative - they should require a code change. This is asking for a comment to be added, which is a minor documentation improvement but not a critical code issue. The existing comment might only apply to the placeholder style based on its position, and the new gridTemplateColumns property might be solving a different aspect of the same issue. Adding an inline comment could genuinely help future maintainers understand why this specific CSS property is needed, especially since it's an unusual grid-related fix for an input element. While adding the comment could be helpful, the existing comment already provides a link to the issue where developers can find more context. The proximity of the existing comment (just one line after the input style block) makes it reasonably clear that these changes are related to the same workaround. This is more of a "nice to have" documentation improvement rather than a necessary code change. This comment is asking for additional documentation rather than identifying a code problem. While it could be helpful, there's already a comment referencing the react-select issue nearby. According to the rules, comments should identify clear code changes needed, not request additional documentation or explanations. This falls into the category of an informative/documentation request.
2. app/components/SearchBar.tsx:141
  • Draft comment:
    The PR description mentions resolving an issue regarding transaction history for deleted accounts, yet the changes here only address SearchBar styling. Ensure that either the description is updated or that additional changes addressing the transaction history are included.
  • Reason this comment was not posted:
    Comment was not on a location in the diff, so it can't be submitted as a review comment.
3. app/components/stories/SearchBar.stories.tsx:39
  • Draft comment:
    The MobileContextMenuFix story effectively verifies that the input has a sufficient touch target width and proper pointer-events for interactivity. For further completeness, consider adding tests for keyboard navigation or accessibility of additional controls (like the clear indicator).
  • Reason this comment was not posted:
    Confidence changes required: 50% <= threshold 50% None
4. docs/development.md:15
  • Draft comment:
    The added mobile testing instructions with HTTPS are clear and helpful. As a minor note, consider mentioning that mobile devices (e.g., iOS Safari) might show warnings about self-signed certificates when running in development.
  • Reason this comment was not posted:
    Confidence changes required: 30% <= threshold 50% None

Workflow ID: wflow_NpWjwPJSEtRPVwZJ

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@vercel
Copy link

vercel bot commented Dec 23, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
explorer Ready Ready Preview, Comment Dec 23, 2025 2:50pm

@C0mberry C0mberry changed the title fix: select styles for mobile fix: touch interaction with search on mobile Dec 23, 2025
@C0mberry C0mberry changed the title fix: touch interaction with search on mobile fix: Touch interaction with search on mobile Dec 23, 2025
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.

2 participants