Skip to content
This repository was archived by the owner on Jan 16, 2023. It is now read-only.

Add demo example - #19

Open
ilyamore88 wants to merge 8 commits into
masterfrom
add-demo-example
Open

Add demo example#19
ilyamore88 wants to merge 8 commits into
masterfrom
add-demo-example

Conversation

@ilyamore88

Copy link
Copy Markdown
Member

No description provided.

Comment thread example/src/styles/player.css Outdated
background: var(--color-bg-element);
box-shadow: 0 1px 1px 0 rgba(0, 0, 0, 0.05);
padding: 8px 15px;
font-family: "Roboto";

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.

пропиши лучше один раз в стилях к html

Comment thread example/src/styles/player.css Outdated
border-radius: 3px;
}

&::-moz-range-track {

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.

В хроме есть баг, который не позволяет этого сделать. Если писать селекторы через зяпятую, то тогда стили для кастомных селекторов не применяются.

@ilyamore88
ilyamore88 marked this pull request as ready for review November 11, 2019 18:26
Comment thread example/src/styles/player.css Outdated
@@ -0,0 +1 @@

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.

image

Comment thread example/src/styles/base.css Outdated
-moz-osx-font-smoothing: grayscale;
}

*, *:before, *:after {

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.

Suggested change
*, *:before, *:after {
*, *::before, *::after {

Comment thread example/src/styles/base.css Outdated
@@ -0,0 +1,28 @@
@import "variables.css";

@import url('https://fonts.googleapis.com/css?family=Roboto&display=swap');

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.

плохо загружать шрифты таким образом. Надо в head прописать

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.

3 participants