Godiesc / firefox-one

Firefox Theme for Opera One skin Lovers
Mozilla Public License 2.0
301 stars 8 forks source link

Vivaldifox support #2

Closed Neikon closed 1 year ago

Neikon commented 1 year ago

en one_root.css he cambiado unos colores prefijados por el dimamico

Probablemente haya mejores y mas elegantes formas de hacerlo pero te hago el pull request por si quieres implementarlo o usarlo de base

close #1

Godiesc commented 1 year ago

lo he probado y no se me ve bien, ahora mismo ando en firefox-beta no se si ese sea el problema, creo además que necesitaría modificarlo más ya que sin la extensión el tema debería de mostrarse con los colores por defecto. No se si lo probaste con algún tema específico de base o algo diferente.

Neikon commented 1 year ago

Creo que ya lo tengo solucionado. He movido --lwt-accent-color: #030615 !important; de one_root a one_root_personal y he creado estas líneas para que sea el usuario el que elija

  --auto-general-color:  var(--lwt-accent-color);
  --auto-acent-color-dark: color-mix(in srgb, var(--auto-color) 75%, white);
  --auto-acent-color-ligth: color-mix(in srgb, var(--auto-color) 75%, black); 

y por defecto lo he dejado para que funcione sin vivaldifox

Godiesc commented 1 year ago

he estado pesando en una manera de hacerlo más personalizable con vivaldifox y sería usar toda una sección de :root para cuando se quiera usar esa extension...

Neikon commented 1 year ago

1- typo error , ya esta arreglado 2- serian para meterlas en --acent-color: var(--lwt-toolbarbutton-icon-fill-attention, #484d65); y esta linea a su vez meterla en @media (prefers-color-scheme: dark){ :root{...} para que cambien segun la web 3- estas capturas son de la config de vivaldifox que tengo https://github.com/Neikon/AutoColor-Minimal-Proton/tree/master/VivaldiFox%20config%20screenshots

Recuerda que este pull request es solo por compartir los cambios que serian necesarios para habilitar vivaldifox por si son de tu interes. no lo tomes como una exigencia/peticion para ponerlo ni mucho menos, puedes descartarlo totalmente si quieres. Es más me picó tanto tu tema y estos cambios que acabé haciendo un tema nuevo gracias a la inspiracion que me diste. https://github.com/Neikon/Fox11

Godiesc commented 1 year ago

chévere tu nuevo tema, hay un error en el readme: useschrome: chrome |- useschrome.css |- components |- other files

y pues creo que me lo pensaré en lo de vivaldi-fox, las variables las reutilizo por todo el tema y el tomar un color muy llamativo como el que pone esa extension puede hacer ilegible algunas cosas, así que ya veré si le hago o no compatible, gracias por todo. ❤

Neikon commented 1 year ago

Gracias por avisarme del error. Entiendo lo que dices, es verdad que es un parche y se nota, aunque ni te preocupes por ello. Por otro lado, muchas gracias por compartir con todos este gran tema que es firefox-one