Ticket #654 (closed defeito: fixed)

Opened 15 years ago

Last modified 14 years ago

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

ticket.654.patch Download (1.9 KB) - added by lucas.dacosta 14 years ago.
Correção da assinatura para o texto rico no IE
assinatura2.dataprev.patch Download (2.3 KB) - added by lucas.dacosta 14 years ago.
com verificação se é a assinatura padrão é texto rico
assinatura3.dataprev.patch Download (821 bytes) - added by lucas.dacosta 14 years ago.
Correção quando a assinatura é texto simples

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

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

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

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?

comment:22 in reply to: ↑ 21 Changed 14 years ago by wmerlotto

Replying to niltonneto:

William, poderia replicar no Branch, se possível?

Feito! Revisão [1651].

Note: See TracTickets for help on using tickets.