Skip to content

Commit e2753c4

Browse files
committed
Revert "Adds repo provider icon to active work in home"
This reverts commit f31b51a. (#4387, #4402)
1 parent f31b51a commit e2753c4

File tree

2 files changed

+5
-25
lines changed

2 files changed

+5
-25
lines changed

src/webviews/apps/plus/home/components/active-work.ts

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,6 @@ export class GlActiveWork extends SignalWatcher(LitElement) {
9090
gl-breadcrumbs {
9191
--gl-tooltip-text-transform: none;
9292
}
93-
9493
.heading-branch-breadcrumb {
9594
text-transform: none;
9695
}
@@ -104,9 +103,6 @@ export class GlActiveWork extends SignalWatcher(LitElement) {
104103
@consume({ context: activeOverviewStateContext })
105104
private _activeOverviewState!: ActiveOverviewState;
106105

107-
@state()
108-
private repoCollapsed = true;
109-
110106
get isPro() {
111107
return isSubscriptionTrialOrPaidFromState(this._homeState.subscription.state);
112108
}
@@ -160,13 +156,13 @@ export class GlActiveWork extends SignalWatcher(LitElement) {
160156
return html`
161157
<gl-section ?loading=${isFetching}>
162158
<gl-breadcrumbs slot="heading">
163-
<gl-breadcrumb-item collapsibleState="none" class="heading-repo-breadcrumb"
159+
<gl-breadcrumb-item collapsibleState="collapsed" icon="repo"
164160
><gl-repo-button-group
165161
.repository=${repo}
162+
.icon=${false}
166163
?disabled=${!hasMultipleRepositories}
167164
?hasMultipleRepositories=${hasMultipleRepositories}
168165
.source=${{ source: 'graph' } as const}
169-
?expandable=${true}
170166
@gl-click=${this.onRepositorySelectorClicked}
171167
><span slot="tooltip">
172168
Switch to Another Repository...

src/webviews/apps/shared/components/repo-button-group.ts

Lines changed: 3 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -38,32 +38,19 @@ export class GlRepoButtonGroup extends GlElement {
3838
refButtonBaseStyles,
3939
truncatedButtonStyles,
4040
css`
41-
:host([icons='1']:not([expandable])) {
41+
:host([icons='1']) {
4242
min-width: 7rem;
4343
}
4444
45-
:host([icons='2']:not([expandable])) {
45+
:host([icons='2']) {
4646
min-width: 9.4rem;
4747
}
4848
49-
:host([icons='2'][expandable]) {
50-
min-width: 5.6rem;
51-
}
52-
5349
.indicator-dot {
5450
--gl-indicator-color: green;
5551
--gl-indicator-size: 0.4rem;
5652
margin-left: -0.2rem;
5753
}
58-
59-
/* :host([expandable]) .truncated-button {
60-
transition: max-width 0.3s cubic-bezier(0.25, 1, 0.5, 1);
61-
} */
62-
63-
:host([expandable]:not(:hover, :focus-within)) .truncated-button {
64-
min-width: 0;
65-
max-width: 0;
66-
}
6754
`,
6855
pickerIconStyles,
6956
];
@@ -86,9 +73,6 @@ export class GlRepoButtonGroup extends GlElement {
8673
@property({ type: Object })
8774
source?: Source;
8875

89-
@property({ type: Boolean, reflect: true })
90-
expandable = false;
91-
9276
@property({ type: Number, reflect: true })
9377
get icons() {
9478
if (this.repository?.provider === undefined) return undefined;
@@ -154,7 +138,7 @@ export class GlRepoButtonGroup extends GlElement {
154138
const { provider } = repo;
155139
const connectedIntegration = provider.integration?.connected;
156140

157-
return html`<gl-popover placement="bottom" trigger="hover click focus">
141+
return html`<gl-popover placement="bottom">
158142
<gl-button
159143
slot="anchor"
160144
part="provider-icon"

0 commit comments

Comments
 (0)