Ticket #654 (closed defeito: fixed)
Salvar assinatura em preferencias pelo IE
Reported by: | lucas.dacosta | Owned by: | lucas.dacosta |
---|---|---|---|
Priority: | grave | Milestone: | Expresso 2.0.2 |
Component: | ExpressoMail | Version: | branch 2.0 |
Severity: | Keywords: | assinatura texto rico bug | |
Cc: | niltonneto | WorkGroup: |
Description
Em preferencias do Expresso Mail ir em preferencias selecionar uma opção para a assinatura, texto simples ou rico e depois inserir um texto.
A assinatura no IE 6 e 7 não é salva, pois quando edita o campo aparece vazio. Obs. Quando seleciona SIM na opção: Inserir assinatura automaticamente em novas mensagens. Funcionou se a assinatura é salva no Firefox.
Attachments
Change History
comment:1 Changed 15 years ago by alfredo.vinicius
- Owner changed from eduardo.rlima to alfredo.vinicius
- Status changed from new to assigned
comment:2 Changed 14 years ago by wmerlotto
- Cc niltonneto added; lucas.dacosta removed
- Keywords texto rico bug added
- Status changed from assigned to new
- Owner changed from alfredo.vinicius to lucas.dacosta
- Milestone set to Expresso 2.0.2
Lucas, vários usuários estão reportando, no fórum, problemas com a assinatura em texto rico no IE. Como vc comentou na reunião do comitê que já estava resolvendo isso, estou realocando para vc...
Changed 14 years ago by lucas.dacosta
- attachment ticket.654.patch added
Correção da assinatura para o texto rico no IE
comment:3 Changed 14 years ago by lucas.dacosta
- Status changed from new to assigned
Essa correção ainda não contempla se a assinatura padrão escolhida for Texto Rico. O problema é que o value do select é igual a vazio, não da pra saber se é texto rico ou simples, então eu estava testando com o valor padrão. O problema é que não pode ser html pois o Texto Rico tem o valor html, vou tentar jogar um valor htmlpadrao para ficar com valor diferente. Bom depois eu mando o outro patch corrigindo.
comment:4 Changed 14 years ago by lucas.dacosta
- Status changed from assigned to closed
- Resolution set to testing
Essa correção ainda não contempla se a assinatura padrão escolhida for Texto Rico. O problema é que o value do select é igual a vazio, não da pra saber se é texto rico ou simples, então eu estava testando com o valor padrão. O problema é que não pode ser html pois o Texto Rico tem o valor html, vou tentar jogar um valor htmlpadrao para ficar com valor diferente. Bom depois eu mando o outro patch corrigindo.
comment:5 Changed 14 years ago by niltonneto
- Priority changed from média to grave
William, quando puder aplicar no Trunk eu agradeço. Preciso saber se corrigiu belezinha para aplicar no Branch 2.0, já que é um bug crítico.
comment:6 Changed 14 years ago by lucas.dacosta
Patch acumulativo verificando se a assinatura padrão é Texto Rico.
Changed 14 years ago by lucas.dacosta
- attachment assinatura2.dataprev.patch added
com verificação se é a assinatura padrão é texto rico
comment:7 Changed 14 years ago by wmerlotto
- Status changed from closed to reopened
- Resolution testing deleted
Apliquei este último patch na revisão [1544]. Quem tiver um IE disponível, por favor, teste! Vou deixar este ticket aberto até que esteja realmente resolvido.
comment:8 follow-up: ↓ 9 Changed 14 years ago by lucas.dacosta
Sobre a correção eu gostaria de fazer um comentário.
No IE document.getElementsByName não funciona bem no IE, nesse caso não funcionou para elementos to tipo select (combo box).
http://www.quirksmode.org/dom/w3c_core.html#miscellaneous]Tabela de compatibilidade
Nota-se que no IE a função getElementsByName() está: Incorrect and incomplete. Recomendo que sempre que usado testar no IE.
comment:9 in reply to: ↑ 8 ; follow-up: ↓ 10 Changed 14 years ago by niltonneto
Replying to lucas.dacosta:
Sobre a correção eu gostaria de fazer um comentário.
No IE document.getElementsByName não funciona bem no IE, nesse caso não funcionou para elementos to tipo select (combo box).
http://www.quirksmode.org/dom/w3c_core.html#miscellaneous]Tabela de compatibilidade
Nota-se que no IE a função getElementsByName() está: Incorrect and incomplete. Recomendo que sempre que usado testar no IE.
Será corrigido para IE?
comment:10 in reply to: ↑ 9 Changed 14 years ago by lucas.dacosta
Replying to niltonneto:
Replying to lucas.dacosta:
Sobre a correção eu gostaria de fazer um comentário.
No IE document.getElementsByName não funciona bem no IE, nesse caso não funcionou para elementos to tipo select (combo box).
http://www.quirksmode.org/dom/w3c_core.html#miscellaneous]Tabela de compatibilidade
Nota-se que no IE a função getElementsByName() está: Incorrect and incomplete. Recomendo que sempre que usado testar no IE.
Será corrigido para IE?
Bom nesse patch eu fiz um novo método para contornar o problema, não creio que isso será arrumado pela Microsoft, pois no IE 8 ainda persiste o problema. Então eu só alerto a todos desenvolvedores para tomar cuidado ao utilizar getElementsByName nos códigos do Expresso.
comment:11 Changed 14 years ago by niltonneto
E então, podemos aplicar essa correção no Branch 2.0 (2.0.2)?
comment:12 Changed 14 years ago by wmerlotto
Lucas, vc viu que um usuário reportou um novo problema, após aplicar o patch, no fórum?
comment:13 Changed 14 years ago by lucas.dacosta
Vou ver. Tb já vou deixar assinado para receber respostas do tópico.
Changed 14 years ago by lucas.dacosta
- attachment assinatura3.dataprev.patch added
Correção quando a assinatura é texto simples
comment:14 follow-up: ↓ 15 Changed 14 years ago by lucas.dacosta
A correção do bug enviado por Fabio Pereira no forum http://www.expressolivre.org/html/modules/newbb/viewtopic.php?viewmode=flat&topic_id=1331&forum=5
está no patch: assinatura3.dataprev.patch
comment:15 in reply to: ↑ 14 Changed 14 years ago by wmerlotto
Replying to lucas.dacosta:
está no patch: assinatura3.dataprev.patch
Patch aplicado na revisão [1605].
comment:16 Changed 14 years ago by lucas.dacosta
- Status changed from reopened to closed
- Resolution set to fixed
Acho que já pode ser fechado.
comment:17 Changed 14 years ago by wmerlotto
Mais alguém testou estas alterações para podermos replicá-las no branch2.0?
comment:18 follow-up: ↓ 19 Changed 14 years ago by tatiane.ito
Acabei de testar com a versão que está no Trunk.
comment:19 in reply to: ↑ 18 Changed 14 years ago by wmerlotto
Replying to tatiane.ito:
Acabei de testar com a versão que está no Trunk.
Maravilha! E funcionou corretamente, no IE e no Firefox?
comment:20 Changed 14 years ago by tatiane.ito
Sim funcionou tanto no IE 7 quanto no Firefox 3.0. O problema descrito no fórum por Fabio Pereira foi alterado.
comment:21 follow-up: ↓ 22 Changed 14 years ago by niltonneto
William, poderia replicar no Branch, se possível?