New CSS template for default look #21

abnazhor · 2021-01-20T14:40:34+01:00

abnazhor commented

2021-01-20 14:40:34 +01:00

Created a new CSS template with dark looks and standarized design for every element.

abnazhor added 1 commit 2021-01-20 14:40:35 +01:00

e4e95839f9 New CSS template for default look

sergiotarxz commented

2021-01-20 15:25:12 +01:00

This new rules do not work well with multiple formats selection like the one needed to support this url. https://www.youtube.com/watch?v=rKrGDVuIzxA

But what is supported looks really great.

This new rules do not work well with multiple formats selection like the one needed to support this url. https://www.youtube.com/watch?v=rKrGDVuIzxA But what is supported looks really great.

sergiotarxz commented

2021-01-20 15:40:06 +01:00

Video container causes horizontal scroll on small screens.

sergiotarxz commented

2021-01-20 15:42:10 +01:00

On mobile width I think would be better the #poping-notice being full width and use 40% on bigger screens using a media query.

Something like this was done by:

@media (min-width: 668px) {
	#poping-notice {
		width: 629px;
	}
}

On mobile width I think would be better the #poping-notice being full width and use 40% on bigger screens using a media query. Something like this was done by: ```css @media (min-width: 668px) { #poping-notice { width: 629px; } } ```

sergiotarxz commented

2021-01-20 15:47:18 +01:00

Add yourself to the AUTHORS file so we can contact you in case LICENSE needs to be changed, for example to upgrade to a newer version of AGPL or things like that.

sergiotarxz changed target branch from master to feature/new_look_default_theme

2021-01-20 20:08:15 +01:00

sergiotarxz merged commit 5228ddfe6b into feature/new_look_default_theme

2021-01-20 20:09:48 +01:00

sergiotarxz referenced this issue from a commit

2021-01-20 20:09:48 +01:00

Merge pull request 'New CSS template for default look' (#21) from abnazhor/Peertube-dl:master into feature/new_look_default_theme

Sign in to join this conversation.

No reviewers