Uniswap / Uniswap/interface

Loading spinner disappears at narrow widget

Open Beginner friendly
#3,677 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
TypeScript
Stars
5.5k
Forks
5.4k
PR merge metrics
No merged PRs in 30d

Description

Desktop. At normal window width, an animated spinner svg appears while the app is fetching a quote form the Auto Router

At more narrow window widths, the animated spinner gets hidden. However there is still plenty of space in the swap box to show the spinner. In fact the swap box doesn't change at all at this width, so there's no reason to hide the spinner.

Here's the relevant line of code: https://github.com/Uniswap/interface/blob/main/src/components/swap/SwapDetailsDropdown.tsx#L144

Screen Shot 2022-04-06 at 6 47 33 PM Screen Shot 2022-04-06 at 6 47 24 PM

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in src/components/swap/SwapDetailsDropdown.tsx at line 144 and reproduce the quote-fetching state at normal and narrow desktop widths. Inspect why the animated spinner SVG is hidden at the narrower width, then verify that it remains visible while the Auto Router fetches a quote without changing the swap box layout.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
frontend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.