Permitir o Expresso carregue diversos templates sem perda de funcionalidade

[amuller]

Implementação/estudo sobre os templates no Expresso, utilizando a API do Egroupware. Retirar códigos com caminhos embutidos que impedem o funcionamento.

[niltonneto]

Replying to amuller:

Implementação/estudo sobre os templates no Expresso, utilizando a API do Egroupware. Retirar códigos com caminhos embutidos que impedem o funcionamento.

Essa ocorrência é de extrema importância para esse momento, pois iremos definir nos próximos dias, o novo tema padrão para a versão oficial do Expresso. Dessa forma, quem quiser continuar com o tema atual, ou utilizar outro tema, não terá problema algum.

[amuller]

Os objetivos são:

• Possibilitar que as cores e figuras sejam carregadas nos templates e não nos códigos.
• Se possível evitar que uma modificação no expresso necessite de modificação em todos os templates.

[niltonneto]

Está disponível para testes no sandbox "preparacao_novos_templates".
:

• Implementada preparação do código para tornar flexível os templates em [773]
  

• Adicionado novo template "Foz do Iguaçu" em [774]
  

• Adicionado arquivos faltantes dos commits anteriores [775]

[niltonneto]

• Adicionado novo template "Canyon" em [776] e [777].

[niltonneto]

• Adicionado novo template "Azul" em [778].

[amuller]

Também registrar aqui que o template do setup precisa ser revisto

• Login
• Settings
• Help/Guide?
• About Trac

[niltonneto]

Merge efetuado no Trunk, dividido em duas revisões:
[790] - Modificação para tornar flexível a utilização de múltiplos templates
[791] - Adição de novos templates (Azul, Canyon e Fozi)

[wmerlotto]

Estou utilizando o código do trunk, revisão [794], com o seguinte ambiente:

• Debian Etch, com os pacotes padrões (php, apache, ...);
• Firefox 3.0.10

Notei alguns bugs, como:

1. Ao abrir uma mensagem e clicar no botão apagar (agora na parte de baixo), é retornada a seguinte mensagem: Nenhuma mensagem foi selecionada e a mensagem não é apagada. Ainda é possível apagar a mensagem selecionando-a pela lista;
2. As mensagens de alterta, erro, ..., exibidas no topo do Expresso (no lugar do nome do usuário) ao serem exibidas alteram a formatação da página, deslocando toda lista de e-mails... Era para ser assim mesmo? Essa mensagem de alerta não teria que ter o mesmo tamanho do nome do usuário, para não criar essa movimentação?

Se notar mais alguma coisa, reportarei neste ticket...

[amuller]

Eu testei no Firefox 3.0.1 e 3.0.9 no Windows e nenhum dele apresentou os problemas relatados. Vou continuar testando.

[wmerlotto]

Bom, vc está utilizando o código do trunk mesmo? E mais um detalhe, estou utilizando Ubuntu 9.04.

[niltonneto]

Testei no Trunk e no Companies/Celepar? e realmente os dois itens acontecem. Estou FF 3.0.9 do Ubuntu.

[wmerlotto]

Notei outro problema, mas não sei é realmente relacionado com o templates. Se não for, me avise que abro outro ticket.

Ao abrir um e-mail com imagens, as mesmas são bloqueadas até que seja clicado no botão (ou link) "show images from...". Com o tema "azul" eu clico neste botão e nada acontece, as imagens do e-mail não são carregadas. Com o tema celepar (padrão), o botão nem é "clicavél"...

A plataforma testada ainda é a mesma, ff-3.0.10 no Ubuntu 9.04.

[niltonneto]

Replying to amuller:

Também registrar aqui que o template do setup precisa ser revisto

• Login
• Settings
• Help/Guide?
• About Trac

Implementado em [796].

[niltonneto]

Corrigido em [798] o caminho do arquivo 'folderopen.png' que estava hardcoded.

[niltonneto]

Replying to wmerlotto:

Notei outro problema, mas não sei é realmente relacionado com o templates. Se não for, me avise que abro outro ticket.

Ao abrir um e-mail com imagens, as mesmas são bloqueadas até que seja clicado no botão (ou link) "show images from...". Com o tema "azul" eu clico neste botão e nada acontece, as imagens do e-mail não são carregadas. Com o tema celepar (padrão), o botão nem é "clicavél"...

A plataforma testada ainda é a mesma, ff-3.0.10 no Ubuntu 9.04.

Corrigido em [800], conforme ticket #38.

[wmerlotto]

Replying to niltonneto:

Corrigido em [800], conforme ticket #38.

Realmente esta funcionalidade está funcionando muito bem, inclusive nos dois temas! Obrigado!

[wmerlotto]

Ao acessar a agenda, com o tema azul, é exibido a seguinte mensagem no fim da página:

Template Error: loadfile: num_dias is not a valid handle.
Backtrace: Template::loadfile / Template::subst / Template::parse / uicalendar::output_template_array / uicalendar::footer / include / common::phpgw_footer / uicalendar::week / ExecMethod
Halted.

Com o tema padrão, celepar, esta mensagem não é exibida.Ainda estou utilizando o mesmo ambiente, Ubuntu 9.04 e FF-3.0.10.

[niltonneto]

Corrigido em [803] o caminho da pasta de templates no arquivo "filters.js".

[niltonneto]

Corrigido em [814] o caminho da pasta de templates, além de alguns ajustes nas variáveis de estilo.

[wmerlotto]

Replying to wmerlotto:

Notei alguns bugs, como:

1. Ao abrir uma mensagem e clicar no botão apagar (agora na parte de baixo), é retornada a seguinte mensagem: Nenhuma mensagem foi selecionada e a mensagem não é apagada. Ainda é possível apagar a mensagem selecionando-a pela lista;
2. As mensagens de alterta, erro, ..., exibidas no topo do Expresso (no lugar do nome do usuário) ao serem exibidas alteram a formatação da página, deslocando toda lista de e-mails... Era para ser assim mesmo? Essa mensagem de alerta não teria que ter o mesmo tamanho do nome do usuário, para não criar essa movimentação?

Só para complementar, quando estou visualizando um e-mail, nenhum dos links (apagar, mover, imprimir, marcar como spam, ...) de baixo funciona.

[amuller]

Wilian: o Nilton ta fazendo o sync ainda. Pode ter coisa que faltou

[niltonneto]

Removido em [819] templates que nunca foram utilizados pelo Expresso.

[niltonneto]

Removido em [820] arquivos desnecessários.

[niltonneto]

Replying to wmerlotto:

Replying to wmerlotto:

Notei alguns bugs, como:

1. Ao abrir uma mensagem e clicar no botão apagar (agora na parte de baixo), é retornada a seguinte mensagem: Nenhuma mensagem foi selecionada e a mensagem não é apagada. Ainda é possível apagar a mensagem selecionando-a pela lista;
2. As mensagens de alterta, erro, ..., exibidas no topo do Expresso (no lugar do nome do usuário) ao serem exibidas alteram a formatação da página, deslocando toda lista de e-mails... Era para ser assim mesmo? Essa mensagem de alerta não teria que ter o mesmo tamanho do nome do usuário, para não criar essa movimentação?

Só para complementar, quando estou visualizando um e-mail, nenhum dos links (apagar, mover, imprimir, marcar como spam, ...) de baixo funciona.

Corrigido em [827] os problemas relatados por William. Correção do caminho de algumas imagens.

[wmerlotto]

Replying to niltonneto:

Corrigido em [827] os problemas relatados por William. Correção do caminho de algumas imagens.

Os botões estão funcionando perfeitamente agora. Só o alerta que ainda continua com problemas...

[niltonneto]

Replying to wmerlotto:

Replying to niltonneto:

Corrigido em [827] os problemas relatados por William. Correção do caminho de algumas imagens.

Os botões estão funcionando perfeitamente agora. Só o alerta que ainda continua com problemas...

Você está se referindo ao item 2, que move o layout pra baixo quando algum alerta é exibido né?

[wmerlotto]

Replying to niltonneto:

Você está se referindo ao item 2, que move o layout pra baixo quando algum alerta é exibido né?

Isso mesmo.

[niltonneto]

Replying to wmerlotto:

Replying to niltonneto:

Você está se referindo ao item 2, que move o layout pra baixo quando algum alerta é exibido né?

Isso mesmo.

Foi corrigido em [831].

[niltonneto]

Corrigido em [835] o estilo do link "salvar" mensagem.

[niltonneto]

Corrigido em [838] mais alterações sobre o estilo do link "salvar" mensagem, além da inclusão da função purgeCache().

[wmerlotto]

Replying to niltonneto:

Foi corrigido em [831].

Realmente foi corrigido. Está funcionado muito bem agora.

Mas a agenda ainda apresenta os problemas de template:

Template Error: loadfile: num_dias is not a valid handle.
Backtrace: Template::loadfile / Template::subst / Template::parse / uicalendar::output_template_array 
/ uicalendar::footer / include / common::phpgw_footer / uicalendar::week / ExecMethod
Halted.

Ao adicionar um novo agendamento, as imagens dos botões (adicionar/remover participantes) também não estão aparecendo. A busca por usuários, no LDAP, também não está funcionando. Estou utilizando a revisão [844], com o tema Azul.

[niltonneto]

Mais erros (testado em ambiernte de homologação, apontando pro Trunk):

1 - Os avisos (write_msg) de "Movido, removido, enviado, etc", continuam deslocando para baixo e para cima todo layout quando aparecem e somem.

2 - No template "celepar", quando o cursor está sobre os filtros de relevância "Todas", "Respondidas", etc, no rodapé, o filtro selecionado muda seu css, retirando o sublinhado e negrito.

[niltonneto]

Replying to niltonneto:

Mais erros (testado em ambiernte de homologação, apontando pro Trunk):

1 - Os avisos (write_msg) de "Movido, removido, enviado, etc", continuam deslocando para baixo e para cima todo layout quando aparecem e somem.

2 - No template "celepar", quando o cursor está sobre os filtros de relevância "Todas", "Respondidas", etc, no rodapé, o filtro selecionado muda seu css, retirando o sublinhado e negrito.

Ambiente cliente: Ubuntu 8.04 com Firefox 3.0.10

[amuller]

Replying to niltonneto:

Replying to niltonneto:

Mais erros (testado em ambiernte de homologação, apontando pro Trunk):

1 - Os avisos (write_msg) de "Movido, removido, enviado, etc", continuam deslocando para baixo e para cima todo layout quando aparecem e somem.

resolvido em [848]

[wmerlotto]

E os erros da agenda? Não serão analisados os outros módulos? Este ticket é só para o ExpressMail??

[alexandrecorreia]

Replying to wmerlotto:

E os erros da agenda? Não serão analisados os outros módulos? Este ticket é só para o ExpressMail??

Vou atrelar esse ticket ao Marco Expresso 1.2, já que não é somente ExpressoMail?.

[alexandrecorreia]

Replying to niltonneto:

Replying to amuller:

Também registrar aqui que o template do setup precisa ser revisto

• Login
• Settings
• Help/Guide?
• About Trac

Implementado em [796].

Corrigido em [853] caminho da figura "orange-ball.png" do setup.

[alexandrecorreia]

Reaberto porque ainda existe problema nos demais módulos.

[wmerlotto]

Replying to alexandrecorreia:

Corrigido em [853] caminho da figura "orange-ball.png" do setup.

Me parece (me corrija se estiver errado) que muitas das alterações são relacionadas aos caminhos das imagens. Aí, executei este comando em busca de alterações que ainda devem ser feitas (seguindo sua última alteração):

cd /var/www/expresso
egrep -rin "(\.\.\/phpgwapi\/templates|idots|idsociety|jerryr|literati|prisma)" * | egrep -v "(.svn|setup\/phpgw)"

Espero que facilite... :D

[amuller]

Resolvido problema do calendar em [855] [854]

[wmerlotto]

Pessoal, não sei vcs já viram, mas a página de adição de agendamento não está funcionando corretamente, pois:

1. As imagens dos botões não estão aparecendo;
2. Não é possível buscar por usuários para adicioná-los como participantes (consigo selecionar a organização);
3. Não é possível adicionar participantes externos;

Estou utilizando o código do trunk, na revisão [857], com FF 3.0.10 no Ubuntu 9.04.

[niltonneto]

Replying to wmerlotto:

Pessoal, não sei vcs já viram, mas a página de adição de agendamento não está funcionando corretamente, pois:

1. As imagens dos botões não estão aparecendo;
2. Não é possível buscar por usuários para adicioná-los como participantes (consigo selecionar a organização);
3. Não é possível adicionar participantes externos;

Estou utilizando o código do trunk, na revisão [857], com FF 3.0.10 no Ubuntu 9.04.

Isso ocorre em qualquer template? Eu tive que fazer uma correção em [856], conforme ultimo comentário no ticket #509.

[wmerlotto]

Replying to niltonneto:

Isso ocorre em qualquer template? Eu tive que fazer uma correção em [856], conforme ultimo comentário no ticket #509.

No tema padrão, celepar, está funcionando corretamente mas no tema azul e foz do iguaçu não. Mas já não estava funcionando antes (da alteração [856] ou anterior).

[amuller]

Wilian: r858

[wmerlotto]

Replying to amuller:

Wilian: r858

Amuller, aparentemente está da mesma forma, a tela de inclusão de agendamentos ainda está com problemas. Verifiquei os logs do apache e encontrei estes erros:

[error] File does not exist: /var/www/expresso/calendar/templates/default/js, referer: http://webmail/index.php?menuaction=calendar.uicalendar.add&date=20090520
[error] File does not exist: /var/www/expresso/calendar/templates/default/images/sent.gif, referer: http://webmail/index.php?menuaction=calendar.uicalendar.add&date=20090520
[error] File does not exist: /var/www/expresso/calendar/templates/default/images/add.png, referer: http://webmail/index.php?menuaction=calendar.uicalendar.add&date=20090520
[error] File does not exist: /var/www/expresso/calendar/templates/default/images/rem.png, referer: http://webmail/index.php?menuaction=calendar.uicalendar.add&date=20090520

Se precisar de outras informações, por favor, me avise.

[amuller]

Ta certo faltou coisa no commit. Agora sim com r862

[wmerlotto]

Replying to amuller:

Ta certo faltou coisa no commit. Agora sim com r862

Agora vc removeu imagens demais, inclusive para o tema padrão (celepar). Segue em anexo um screenshot de um agendamento.

Na página de adição de agendamento, a consulta por participantes não está funcionando (somente com os novos temas, azul, fozi, ...).

[wmerlotto]

Screenshot de um agendamento. Alguns ícones não estão sendo carregados, tanto no tema antigo (celepar) quanto nos novos.

[rafaelraymundo]

Replying to wmerlotto:

Replying to amuller:

Ta certo faltou coisa no commit. Agora sim com r862

Agora vc removeu imagens demais, inclusive para o tema padrão (celepar). Segue em anexo um screenshot de um agendamento.

Na página de adição de agendamento, a consulta por participantes não está funcionando (somente com os novos temas, azul, fozi, ...).

Usando o template Serpro, com o calendar da comunidade unificado percebi que a revisão #858 que retira códigos amarrados a um template quebram os botões de Adicionar/Remover? e Avisar Partic Externos. Caso essas linhas usem GLOBALS ao ivés de SESSION esse erro não ocorre. No cód abaixo referente a "class.uicalendar.inc.php" ele retorna o botão Adicionar corretamente.

'<button type="button" onClick="javascript:add_user();"><img src="calendar/templates/'.$GLOBALSphpgw_info?user?preferences?common?template_set?.'/images/add.png" style="vertical-align: middle;" >&nbsp;'.lang("Add").'</button>'. ...

Na implementação da revisão #858 da forma abaixo, os botões == não carregam == corretamente

... '<button type="button" onClick="javascript:add_user();"><img src="calendar/templates/'.$_SESSIONphpgw_info?calendaruser?preferences?common?template_set?.'/images/add.png" style="vertical-align: middle;" >&nbsp;'.lang("Add").'</button>'.

>>

Mas caso permaneça como GLOBALS isso pode ter algum impacto nos templates que estão sendo implementados.

[rafaelraymundo]

Corrigido em [933] - Código alterado de Globals para Session, mas os links das três imagens comentados anteriormente permanecem como Globals

[amuller]

As revisões 938 939 940 941 944 945 retiram os templates e transformam em temas e adiciona o login azul feito por designers da celepar

[rafaelraymundo]

Muller, Essas alterações no login impactaram nas nossas modificações. Necessitamos separar a lógica dos logins e commitar no trunk da comunidade, para evitar problemas desse tipo.

ok?

[rafaelraymundo]

Corrigido em [948] [949] - Lógica do login foi separada em: /phpgwapi/templates/celepar/login_celepar.php /phpgwapi/templates/azul/login_azul.php /phpgwapi/templates/default/login_default.php

expresso/login.php identifica qual template utilizar

[niltonneto]

Corrigido em [953] [954] rodapé do login azul (template e imagens).

[rafaelraymundo]

É interessante prever que da forma que está a caixa de login não será expandida na inserção de um captcha e de sua caixa de texto. O ideal seria que essa caixa de login fosse uma div com uma imagem de fundo.

[niltonneto]

Replying to rafaelraymundo:

É interessante prever que da forma que está a caixa de login não será expandida na inserção de um captcha e de sua caixa de texto. O ideal seria que essa caixa de login fosse uma div com uma imagem de fundo.

Concordo contigo. O tamanho da caixa tinha que ser dinâmico, mas o WebDesign? encarregado desse novo layout entregou dessa forma pra nós. Vou verificar com ele.

[amuller]

commit 963 implementa caixa com tamanho variável

[niltonneto]

1 - Adaptar código javascript para suportar duas classes CSS (tr_msg_read e tr_msg_read2) na grade da lista de mensagens. O objetivo é tornar o produto final bem perto do layout proposto. Para não impactar nos demais templates, como o atual, será acresentado um alias 'tr_msg_read2' na mesma classe CSS 'tr_msg_read'.

2 - Adaptar resultado da busca com mesmo formato.

3 - Importante: Por padrão, as linhas da lista de mensagens terão sempre como CSS 'tr_msg_read' e 'tr_msg_read2', alternadamente. Para mensagems não-lidas, será acrescentada a classe 'tr_msg_unread'. Para mensagens importantes, a classe 'tr_flagged_msg'. E por fim, para mensagens selecionadas, a classe 'selected_msg'.

[niltonneto]

Replying to niltonneto:

1 - Adaptar código javascript para suportar duas classes CSS (tr_msg_read e tr_msg_read2) na grade da lista de mensagens. O objetivo é tornar o produto final bem perto do layout proposto. Para não impactar nos demais templates, como o atual, será acresentado um alias 'tr_msg_read2' na mesma classe CSS 'tr_msg_read'.

2 - Adaptar resultado da busca com mesmo formato.

3 - Importante: Por padrão, as linhas da lista de mensagens terão sempre como CSS 'tr_msg_read' e 'tr_msg_read2', alternadamente. Para mensagems não-lidas, será acrescentada a classe 'tr_msg_unread'. Para mensagens importantes, a classe 'tr_flagged_msg'. E por fim, para mensagens selecionadas, a classe 'selected_msg'.

Implementado em [976]

[niltonneto]

Corrigido em [992] problema na figura de fundo da div "content_folders" no template azul.

[niltonneto]

Corrigido em [993] problema da borda do checkbox, que sempre aparece no Internet Explorer.

[niltonneto]

Revertido em [994] codificação das flags das mensagens na lista draw_box, devido à problemas de transparência das imagens.

[amuller]

Melhorias [1049] [1054] [1055]

[amuller]

phpgwapi/temas

Página criada para esclarecimento.

[niltonneto]

Adicionado em [1077:1081] template azul no módulo Ajuda.

[niltonneto]

Replying to niltonneto:

Adicionado em [1077:1081] template azul no módulo Ajuda.

 Arquivos e pastas adicionadas.

[niltonneto]

Modificado em [1083] o template do setup.

[niltonneto]

Modificado em  [1093:1096]a ajustes na página inicial.

[niltonneto]

Alterado em [1100] toda implementação referente link "Sugestões" do Expresso, para usar multiplos templates. Além disso, foi criado template azul para o mesmo. Correções no arquivo de idioma pt-BR.

[amuller]

[1108]

[amuller]

Correção do logo no ie [1133]

[niltonneto]

AlexandreMuller?, favor fazer as devidas modificações para alterarmos as pastas templates "azul" para "default", e de "celepar" para "classico". Lembre-se que os links simbólicos também deverão ser alterados dentro do "azul", já que apontam para o "celepar".

[amuller]

r1151 implementa a mudança

[niltonneto]

O link "Fonte da Mensagem" não existe mais no novo template. Muitos usuários terão dificuldade em "descobrir" que o link "Exportar" de dentro da mensagem faz a mesma coisa. Favor recolocar o link ao lado de "Responder a todos sem histórico".

[amuller]

[1208] Volta com dois links na parte superior e inferior.

[niltonneto]

Replying to amuller:

[1208] Volta com dois links na parte superior e inferior.

Replicado no Branch 2.0 em [1221].

[niltonneto]

O fundo do ícone do módulo atual não está em destaque quando é carregado. Isto é percebido nos novos temas, já que possuem imagem diferente na classe 'navbar_butOut'.

[niltonneto]

Replying to niltonneto:

O fundo do ícone do módulo atual não está em destaque quando é carregado. Isto é percebido nos novos temas, já que possuem imagem diferente na classe 'navbar_butOut'.

Corrigido no Trunk em [1215]. Replicado no Branch 2.0 em [1222].

[niltonneto]

Imagens quebradas ao visualizar status do compromisso da agenda, conforme o  link dessa imagem. Muller, se puder verificar, agradeço.

[niltonneto]

Replying to niltonneto:

Alterado em [1100] toda implementação referente link "Sugestões" do Expresso, para usar multiplos templates. Além disso, foi criado template azul para o mesmo. Correções no arquivo de idioma pt-BR.

Para os links "Ajuda" e "Sugestões", as pastas "azul" e "celepar" foram movidas para "default" e "classic" respectivamente. Corrigido em  1248 a 1253.

[niltonneto]

Replying to niltonneto:

Imagens quebradas ao visualizar status do compromisso da agenda, conforme o  link dessa imagem. Muller, se puder verificar, agradeço.

Corrigido em [1306].

[amuller]

Correção de problema da função fora de escritório ocasionado por remoção de código das preferências [1308]

[niltonneto]

Replying to amuller:

Correção de problema da função fora de escritório ocasionado por remoção de código das preferências [1308]

Esse problema se refere ao ticket #485.

[niltonneto]

Podemos fechar esse ticket?

[niltonneto]

Corrigido em [1325] problema do campo texto rico, referente ao termo de aceito no Admin.

[niltonneto]

Replying to niltonneto:

Podemos fechar esse ticket?

OK, acredito que não temos mais nenhum problema grave em relação a essa implementação.