Ticket #987 (closed defeito: fixed)

Opened 11 years ago

Last modified 11 years ago

Carregar personalização de tema por módulo

Reported by: rodsouza Owned by: rodsouza
Priority: grave Milestone: Expresso 2.0.5
Component: API Version: branch 2.0
Severity: Keywords: temas módulos personalização
Cc: WorkGroup:

Description (last modified by rodsouza) (diff)

Módulos como o administrador, preferências e o workflow não são apresentados corretamente quando o tema é alterado.

Nos módulos administrador e preferências tornam-se inutilizáveis.

O módulo worflow simplesmente não respeita os temas.

Ambas situações ocorrem devido a impossibilidade de realizar alterações no tema específicas aos módulos.

Para possibilitar essa personalização e a correção do problema em questão a API carregará o arquivo em questão caso exista.

Vide ticket #987 e #992.

Attachments

antes_depois.jpg Download (657.3 KB) - added by niltonneto 11 years ago.
Diferenças após commit no template clássico (ExpressoMail?).

Change History

comment:1 Changed 11 years ago by rodsouza

Adicionada a possibilidade de personalizar o tema por módulo.

phpgwapi/inc/class.common.inc.php

Committed revision r2269.

comment:2 Changed 11 years ago by rodsouza

Unificando local da inclusão do css do tema.

phpgwapi/inc/class.common.inc.php
phpgwapi/templates/classic/head.tpl
phpgwapi/templates/default/head.inc.php
phpgwapi/templates/default/head.tpl

Committed revision r2273.

comment:3 Changed 11 years ago by rodsouza

A ordem de inclusão do css é a seguinte:

  • <tema>.css (padrão)
  • app.css (módulo)
  • <tema>.css (personalizado - módulo)

comment:4 Changed 11 years ago by rodsouza

Ordenando de inclusão dos estilos padrões quanto a relevância, ou seja:

  • <tema>.css (padrão)
  • <tema>.css (personalizado - módulo)
  • app.css (módulo)
phpgwapi/inc/class.common.inc.php

Committed revision r2274.

comment:5 Changed 11 years ago by rodsouza

Removendo criação de log desnecessária.

phpgwapi/inc/class.common.inc.php

Committed revision r2315.

comment:6 follow-up: ↓ 7 Changed 11 years ago by niltonneto

Rodrigo, só um lembrete. Quando finalizar esse ticket, por favor, replique sua correção nos branches 2.0 e 2.1.

comment:7 in reply to: ↑ 6 ; follow-up: ↓ 8 Changed 11 years ago by wmerlotto

  • Description modified (diff)

Replying to niltonneto:

Rodrigo, só um lembrete. Quando finalizar esse ticket, por favor, replique sua correção nos branches 2.0 e 2.1.

Este tipo de alteração irá para o branch2.0? É realmente necessário?

comment:8 in reply to: ↑ 7 Changed 11 years ago by niltonneto

Replying to wmerlotto:

Replying to niltonneto:

Rodrigo, só um lembrete. Quando finalizar esse ticket, por favor, replique sua correção nos branches 2.0 e 2.1.

Este tipo de alteração irá para o branch2.0? É realmente necessário?

Com certeza. A versão 2.0 é a versão oficial e a mais estável do projeto. Além disso, o Rodrigo está corrigindo esse problema principalmente porque aqui na Celepar temos a 2.0 em produção.

comment:9 follow-up: ↓ 11 Changed 11 years ago by rodsouza

Replicada a correção no branche 2.0

Committed revision r2383.

comment:10 Changed 11 years ago by rodsouza

Replicada a correção no branche 2.1

Committed revision r2385.

comment:11 in reply to: ↑ 9 Changed 11 years ago by niltonneto

Replying to rodsouza:

Replicada a correção no branche 2.0

Committed revision r2383.

Após esse commit, o template clássico ficou com alguns desajustes no ExpressoMail? (anexo antes_depois.jpg).

Changed 11 years ago by niltonneto

Diferenças após commit no template clássico (ExpressoMail?).

comment:12 Changed 11 years ago by niltonneto

  • Version changed from trunk to branch 2.0
  • Milestone changed from Expresso 2.2 to Expresso 2.0.5

comment:13 Changed 11 years ago by niltonneto

Logo sumiu no template default, tema azul => File does not exist: /var/www/expresso-2.0/phpgwapi/templates/default/images/logo_expresso.jpg, referer:  http://localhost/expresso-2.0/phpgwapi/templates/default/css/azul.css

comment:14 follow-up: ↓ 15 Changed 11 years ago by rodsouza

O problema ocorre devido a utilização do formato "jpg" e "png".

Pois a imagem existe em ambos os formatos porém em locais diferentes. Sugiro que seja utilizado o formato "jpg".

comment:15 in reply to: ↑ 14 Changed 11 years ago by niltonneto

Replying to rodsouza:

O problema ocorre devido a utilização do formato "jpg" e "png".

Pois a imagem existe em ambos os formatos porém em locais diferentes. Sugiro que seja utilizado o formato "jpg".

Favor efetuar a respectiva correção então.

comment:16 Changed 11 years ago by rodsouza

Alterando o formato da imagem referente ao logo do Expresso para JPG.

phpgwapi/templates/classic/navbar.tpl
phpgwapi/templates/default/images/logo_expresso.jpg (Adding)
phpgwapi/templates/default/images/logo_expresso.png (Deleting)

Committed revision r2395.

comment:17 Changed 11 years ago by rodsouza

Corrigindo a logo do ExpressoLivre? que não era encontrada. Branche 2.0.

phpgwapi/templates/classic/navbar.tpl

Committed revision r2412.

comment:18 Changed 11 years ago by rodsouza

Corrigindo cores de linhas alternadas no tema celepar.

phpgwapi/templates/classic/css/celepar.css
phpgwapi/templates/default/css/celepar.css

Committed revision r2432.

comment:19 follow-up: ↓ 26 Changed 11 years ago by rodsouza

Corrigindo a exibição do nome 'Expresso Mail' localizado em 'divAppboxHeader'.

expressoMail1_2/templates/classic/main.css
expressoMail1_2/templates/default/main.css

Committed revision r2433.

comment:20 Changed 11 years ago by rodsouza

Removendo o itálico do nome localizado em 'divAppboxHeader' no template clássico.

phpgwapi/templates/classic/css/celepar.css

Committed revision r2434.

comment:21 Changed 11 years ago by rodsouza

Corrigindo alguns problemas com formatação da fonte no tema clássico."

phpgwapi/templates/classic/css/celepar.css

Committed revision r2435.

expressoMail1_2/templates/classic/main.css
expressoMail1_2/templates/default/main.css
phpgwapi/templates/default/css/celepar.css

Committed revision r2436.

comment:22 Changed 11 years ago by rodsouza

Corrigindo problemas com cores de linhas alternadas no tema clássico. (trunk)

phpgwapi/templates/classic/css/celepar.css
phpgwapi/templates/default/css/celepar.css

Committed revision r2438.

comment:23 follow-up: ↓ 28 Changed 11 years ago by niltonneto

Corrigido no Branch 2.0 [2441] alguns problemas de estilo no template classic.

comment:24 follow-up: ↓ 27 Changed 11 years ago by niltonneto

Corrigido no Branch 2.0 [2443] 'font-variant' do nome em 'divAppboxHeader' no template classic. Sobrescreve o valor padrão em base.css

comment:25 follow-up: ↓ 29 Changed 11 years ago by niltonneto

Removido em Branch 2.0 [2468] o tipo de fonte das abas no template default.

comment:26 in reply to: ↑ 19 Changed 11 years ago by niltonneto

Replying to rodsouza:

Corrigindo a exibição do nome 'Expresso Mail' localizado em 'divAppboxHeader'.

expressoMail1_2/templates/classic/main.css
expressoMail1_2/templates/default/main.css

Committed revision r2433.

Replicada correção no template classic, branch 2.1 [2499] e Trunk [2498].

comment:27 in reply to: ↑ 24 Changed 11 years ago by niltonneto

Replying to niltonneto:

Corrigido no Branch 2.0 [2443] 'font-variant' do nome em 'divAppboxHeader' no template classic. Sobrescreve o valor padrão em base.css

Revisões [2434][2443] foram replicadas no Branch 2.1 [2500] e Trunk [2501].

comment:28 in reply to: ↑ 23 Changed 11 years ago by niltonneto

Replying to niltonneto:

Corrigido no Branch 2.0 [2441] alguns problemas de estilo no template classic.

Replicado no Branch 2.1 [2502] e Trunk [2503]

comment:29 in reply to: ↑ 25 Changed 11 years ago by niltonneto

Replying to niltonneto:

Removido em Branch 2.0 [2468] o tipo de fonte das abas no template default.

Replicado no Branch 2.1 [2504] e Trunk [2506]

comment:30 Changed 11 years ago by niltonneto

Acredito que todos os problemas deste ticket foram corrigidos no Trunk, Branch 2.0 e 2.1. Vou deixar até amanhã aberto, caso alguém tenha encontrado algo.

comment:31 Changed 11 years ago by rodsouza

  • Owner changed from alguem to rodsouza

Existe alguns problemas de layout mas estão espalhados e ainda tem a necessidade de realizar o trabalho feito nos css azul, cataratas e rochas nos demais temas, entretanto isso poder ser tema de outro ticket.

comment:32 follow-up: ↓ 34 Changed 11 years ago by rodsouza

Na verdade as informações de compatibilidade contidas nesse ticket deveriam estar em #989.

Finalizando o ticket por esse motivo.

comment:33 Changed 11 years ago by rodsouza

  • Status changed from new to closed
  • Resolution set to fixed

comment:34 in reply to: ↑ 32 Changed 11 years ago by rodsouza

Replying to rodsouza:

Na verdade as informações de compatibilidade contidas nesse ticket deveriam estar em #989.

Finalizando o ticket por esse motivo.

O ticket referente a essa compatibilização não é #989 e sim #992.

comment:35 Changed 11 years ago by rodsouza

  • Description modified (diff)
Note: See TracTickets for help on using tickets.