Skip to content
This repository has been archived by the owner on Aug 1, 2022. It is now read-only.

fix(ui): move go to profile to isExperimental #1261

Merged
merged 4 commits into from Nov 17, 2020
Merged

Conversation

juliendonck
Copy link
Member

Because we can't resolve the profiles of contributors yet (only of the maintainer) from the peerselector, we've decided to move it to isExperimental for now.

Closes #1254

@juliendonck juliendonck added the ui label Nov 17, 2020
@juliendonck juliendonck added this to the β1 milestone Nov 17, 2020
@juliendonck juliendonck self-assigned this Nov 17, 2020
@juliendonck juliendonck requested review from rudolfs and MeBrei and removed request for rudolfs November 17, 2020 09:22
@juliendonck juliendonck changed the title feat(ui): move go to profile to isExperimental fix(ui): move go to profile to isExperimental Nov 17, 2020
@xla xla removed this from the β1 milestone Nov 17, 2020
@juliendonck juliendonck added this to the β1 milestone Nov 17, 2020
xla
xla previously approved these changes Nov 17, 2020
Copy link
Contributor

@xla xla left a comment

Choose a reason for hiding this comment

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

🕹 ⬇️ 😶 😲

FintanH
FintanH previously approved these changes Nov 17, 2020
Copy link
Contributor

@FintanH FintanH left a comment

Choose a reason for hiding this comment

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

You the real MVP

@rudolfs
Copy link
Member

rudolfs commented Nov 17, 2020

I wonder if we should show the collapse icon in the expanded dropdown as well to prevent it from "jumping" around?

current suggestion
jump no-jump
diff --git a/ui/Screen/Project/PeerSelector.svelte b/ui/Screen/Project/PeerSelector.svelte
index 791d80a1..2d2a33ba 100644
--- a/ui/Screen/Project/PeerSelector.svelte
+++ b/ui/Screen/Project/PeerSelector.svelte
@@ -217,6 +217,9 @@
                 <Icon.ArrowBoxUpRight />
               </div>
             </Tooltip>
+          {:else}
+              <Icon.ChevronUpDown
+                style="margin-left: 0.5rem; vertical-align: bottom; fill: var(--color-foreground-level-4)" />
           {/if}
         </div>
       {/each}

@juliendonck juliendonck dismissed stale reviews from FintanH and xla via 1194144 November 17, 2020 10:21
@juliendonck juliendonck merged commit 37966ed into master Nov 17, 2020
@juliendonck juliendonck deleted the julien/go-to-profile branch November 17, 2020 13:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move "go to profile" to isExperimental
5 participants