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

chore: remove bright counter from sidebar tab selector #31975

Merged
merged 2 commits into from
Aug 1, 2024

Conversation

yury-s
Copy link
Member

@yury-s yury-s commented Aug 1, 2024

Removing the following icon:
image

HTML options element cannot be styled, so just removing the counter in sidebar mode:

image

suffix = ` 🔵✖️${tab.count}`;
if (tab.errorCount === 1)
suffix = ` 🔴`;
else if (tab.errorCount)
Copy link
Member

Choose a reason for hiding this comment

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

We seem to have this code in a regular tabbed pane mode.

This comment has been minimized.

Copy link
Contributor

github-actions bot commented Aug 1, 2024

Test results for "tests 1"

1 failed
❌ [installation tests] › playwright-component-testing.spec.ts:55:5 › pnpm: JSX inside a @playwright/test should work

5 flaky ⚠️ [firefox-library] › library/browsercontext-fetch-happy-eyeballs.spec.ts:46:3 › get should work
⚠️ [firefox-page] › page/page-goto.spec.ts:182:3 › should properly cancel Cross-Origin-Opener-Policy navigation
⚠️ [playwright-test] › ui-mode-test-setup.spec.ts:87:5 › should show errors in config
⚠️ [playwright-test] › ui-mode-test-attachments.spec.ts:102:5 › should linkify string attachments
⚠️ [webkit-page] › page/workers.spec.ts:243:3 › should support offline

29522 passed, 702 skipped
✔️✔️✔️

Merge workflow run.

@yury-s yury-s merged commit 5a80ddf into microsoft:main Aug 1, 2024
29 of 30 checks passed
@yury-s yury-s deleted the no-bright-icons-sidebar branch August 1, 2024 23:18
yury-s added a commit to yury-s/playwright that referenced this pull request Aug 1, 2024
…ar tab selector

Removing the following icon:

![image](https://github.com/user-attachments/assets/d2de2ed0-f66e-4452-8763-aad1b6e7bb79)

HTML `options` element cannot be styled, so just removing the counter in
sidebar mode:

<img width="348" alt="image"
src="https://github.com/user-attachments/assets/d636dca2-5007-41f7-866e-3a0f604d46fc">
yury-s added a commit that referenced this pull request Aug 2, 2024
#31976)

…lector

Removing the following icon:


![image](https://github.com/user-attachments/assets/d2de2ed0-f66e-4452-8763-aad1b6e7bb79)

HTML `options` element cannot be styled, so just removing the counter in
sidebar mode:

<img width="348" alt="image"

src="https://github.com/user-attachments/assets/d636dca2-5007-41f7-866e-3a0f604d46fc">
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.

3 participants