Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[N6] AddressSearch component - Border lingers after close #5810

Closed
aldo-expensify opened this issue Oct 13, 2021 · 8 comments
Closed

[N6] AddressSearch component - Border lingers after close #5810

aldo-expensify opened this issue Oct 13, 2021 · 8 comments
Assignees
Labels
Engineering Improvement Item broken or needs improvement. Reviewing Has a PR in review Weekly KSv2

Comments

@aldo-expensify
Copy link
Contributor

aldo-expensify commented Oct 13, 2021

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Action Performed:

  1. Create NewDot account, verify
  2. Navigate to /bank-account/company
  3. Option 1:
    1. Type something that doesn't yield any result by the auto-complete (i.e. asdasdas)
    2. The list of results is empty and we get the grey line
  4. Option 2:
    1. Type something that yield results
    2. Select any result
    3. Click again on the input (focus)

Expected Result:

  • When there are no results to display, we should have a grey line
  • Focusing on the input after selecting a result should make the grey line appear

Actual Result:

There is a grey line below the field

Screen Shot 2021-10-13 at 10 49 00 AM

Screen.Recording.2021-10-13.at.10.48.43.AM.mov

Workaround:

No workaround necessary - just styling stuff

Platform:

Where is this issue occurring?

  • Web
  • iOS
  • Android
  • Desktop App
  • Mobile Web

Version Number:
Reproducible in staging?:
Reproducible in production?:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:

View all open jobs on GitHub

@aldo-expensify aldo-expensify added AutoAssignerTriage Auto assign issues for triage to an available triage team member Daily KSv2 labels Oct 13, 2021
@MelvinBot
Copy link

Triggered auto assignment to @cdraeger11 (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

@MelvinBot MelvinBot removed the AutoAssignerTriage Auto assign issues for triage to an available triage team member label Oct 13, 2021
@Luke9389
Copy link
Contributor

@cdraeger11 no need to triage this. We'll work on it internally (deadline is Friday so External's not gonna cut it)

@MelvinBot
Copy link

Huh... This is 4 days overdue. Who can take care of this?

@MelvinBot
Copy link

Still overdue 6 days?! Let's take care of this!

@Luke9389 Luke9389 added Weekly KSv2 and removed Daily KSv2 labels Oct 21, 2021
@MelvinBot MelvinBot removed the Overdue label Oct 21, 2021
@luacmartins
Copy link
Contributor

I don't see this issue occurring anymore. Should we close this?

@aldo-expensify
Copy link
Contributor Author

I don't see this issue occurring anymore. Should we close this?

It is still there, you have to select a result then click on the input again

Screen Shot 2021-10-28 at 2 32 00 PM

Updated steps to reproduce

@luacmartins
Copy link
Contributor

luacmartins commented Oct 29, 2021

Looking at this comment FaridSafi/react-native-google-places-autocomplete#769 (comment), I'm not sure that rendering a fragment would solve the problem.

We can render a fragment or any other component like this listEmptyComponent={() => <></>}, but they are rendered inside the listView, which already has the borders and the problem persists.

I put up a PR with a workaround for now.

@luacmartins luacmartins added Improvement Item broken or needs improvement. Engineering labels Oct 29, 2021
@aldo-expensify
Copy link
Contributor Author

Looking at this comment FaridSafi/react-native-google-places-autocomplete#769 (comment), I'm not sure that rendering a fragment would solve the problem.

We can render a fragment or any other component like this listEmptyComponent={() => <></>}, but they are rendered inside the listView, which already has the borders and the problem persists.

I put up a PR with a workaround for now.

I think you are right about this, I tried something like this (return null or I don't remember well), but it will still render the box with borders.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering Improvement Item broken or needs improvement. Reviewing Has a PR in review Weekly KSv2
Projects
None yet
Development

No branches or pull requests

6 participants