Inicio » Trisquel » Incidencias
Enviado por Legimet o Sáb, 01/11/2014 - 16:49

Abrowser looks ugly in KDE

Proxecto:Trisquel
Versión:7.0
Componente:Packages
Categoría:informe de erro
Prioridade:normal
Asignado:Legimet
Estado:closed
Descrición

Abrowser looks ugly in KDE with the gtk2-engines-oxygen package. Apparently this package does some special things for XUL apps such as Firefox and Thunderbird.

An easy fix is as follows:
In the gtk2-engines-oxygen package, go to src/oxygenapplicationname.cpp, and insert the string "abrowser" into the array XulAppNames (at line 74). This fixed the problem for me.

I have attached a screenshot of Abrowser before patching gtk2-engines-oxygen, and after patching it.

AnexoTamaño
before.png360.26 KB
after.png355.91 KB
Legimet:
Xov, 01/16/2014 - 19:13

The following line should work for a package helper:
sed '76i \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \"abrowser\"\,' -i src/oxygenapplicationname.cpp

Legimet:
Mar, 01/21/2014 - 01:44
Estado:active» patch (needs review)
BugRep:
Mér, 01/22/2014 - 00:05

If some lines were to be added to the file before the 76th line by upstream, the file would have errors. It would be better not to use hardcoded line numbers.

FILE=src/oxygenapplicationname.cpp
LINENUM=$(grep -n '"firefox",' $FILE | cut -d: -f1)
sed -i $LINENUM'i \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \"abrowser\"\,' $FILE
Legimet:
Ven, 04/18/2014 - 21:21
Asignado a:anonymous» Legimet

A patch for this can be found here.

Legimet:
Mér, 05/28/2014 - 02:03

With the upcoming change from Abrowser to Icecat, this patch won't be necessary because "icecat" is already in that array. Once Trisquel switches to IceCat I'll close this issue.

quidam:
Mér, 05/28/2014 - 02:56
Estado:patch (needs review)» fixed

Added to helpers and available in the repos now. Sorry for the delay, and thanks for the contribution!

quidam:
Mér, 06/11/2014 - 03:00
Estado:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

Legimet:
Ven, 12/05/2014 - 01:20
Versión:6.0» 7.0
Estado:closed» patch (needs review)

I've been using the version in my PPA, but I just realized that it needs to fixed in Trisquel 7 as well (since Abrowser is still being used).

Legimet:
Ven, 12/05/2014 - 18:15

Fixed.

SirGrant:
Dom, 12/14/2014 - 14:23
Estado:patch (needs review)» fixed
quidam:
Dom, 12/28/2014 - 14:25
Estado:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.