Skip to content

Restore feature: Remove most tooltips#799

Merged
fregante merged 2 commits into
masterfrom
nay-tooltips
Nov 9, 2017
Merged

Restore feature: Remove most tooltips#799
fregante merged 2 commits into
masterfrom
nay-tooltips

Conversation

@fregante
Copy link
Copy Markdown
Member

@fregante fregante commented Nov 6, 2017

Closes #731
Fixes #689

@fregante fregante added the bug label Nov 6, 2017
Comment thread extension/content.css
.avatar:before,
.avatar:after {
display: inline-block !important;
}
Copy link
Copy Markdown
Member Author

@fregante fregante Nov 6, 2017

Choose a reason for hiding this comment

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

Sadly:

  • :matches isn't available
  • CSS Nesting module isn't available

Either one of them would shorten these selectors considerably.

  • :not(a):not(b) can't be broken into multiple lines
  • :not(a, b) isn't available

These at least would limit them to one rule.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think you can use :-webkit-any(a, b) instead of :matches for now.

See: https://css-tricks.com/almanac/selectors/m/matches/

Copy link
Copy Markdown
Member Author

@fregante fregante Nov 6, 2017

Choose a reason for hiding this comment

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

But, Firefox.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes. That requires two separate rules though, so we solve nothing 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants