Ticket #331 (closed defeito: fixed)

Opened 12 years ago

Last modified 9 years ago

Teclas de atalho;

Reported by: jakjr Owned by: jakjr
Priority: grave Milestone:
Component: ExpressoMail Version: 1.0
Severity: Keywords: teclas atalho seleção
Cc: WorkGroup:

Description

Como alguém mexeu no script de teclas de atalho e não testou direito, agora, ao apertar a tecla shift + seta para baixo ou para cima, ele não seleciona as mensagens e sim, seleciona todos os elementos da página, como se fosse o mouse. Favor corrigir e testar melhor antes de mexer em algo que outros fizeram ...

Attachments

erro.png Download (169.0 KB) - added by jakjr 12 years ago.

Change History

comment:1 follow-up: ↓ 3 Changed 12 years ago by jakjr

  • Milestone set to Expresso Mail 1.232

comment:2 Changed 12 years ago by amuller

  • Keywords teclas seleção added; defeito removed

Eu testei no meu Expresso local e produção e funcionou. Foi mexido aqui?

Changed 12 years ago by jakjr

comment:3 in reply to: ↑ 1 Changed 12 years ago by jakjr

Replying to jakjr: Ver anexo.

comment:4 Changed 12 years ago by rodsouza

Realizado teste no Firefox(Iceweasel) 2.0.0.3 e o fato relatado não acontece.

Quando as teclas direcionais são combinadas com a tecla shift o navegador dispara apenas um evento e não mais a cadeia de eventos que ocorre quando a tecla shift não está pressionada, todavia as mensagens são corretamente selecioandas.


Realizado teste no Firefox(Iceweasel) 2.0.0.15 e o fato relatado não acontece.

O resultado desse teste foi semelhante ao obtido no Firefox(Iceweasel) 2.0.0.15.


Realizado teste no Firefox 3.0.1 e o fato relatado não acontece.

Quando as teclas direcionais são combinadas com a tecla shift o navegador dispara a cadeia de eventos (diferentemente do ocorrido no teste realizado na versão 2.0.0.3) e as mensagens são selecionadas corretamente.


Realizado teste no Internet Explorer 6.0.29 e o fato relatado não acontece.

O resultado desse teste foi semelhante ao obtido no Firefox 3.0.1. Apenas ressaltando uma ressalva, neste caso são selecionadas duas mensagens ao invés de uma (por vez).

comment:5 Changed 12 years ago by rodsouza

Após verificar o histórico de alterações, consta que:

  • todas as alterações foram feitas pelo mesmo desenvolvedor;
  • nenhuma das alterações interferem diretamente na funcionalidade em questão;

Assim sendo "a priori" o fato relatado possui outra causa, ou ainda é específico a um contexto, não sendo causado por alterações.

comment:6 Changed 12 years ago by rodsouza

Após inúmeros teste constatou-se:

  • o fato em questão não é causado por alteração no código;
  • o fato existe desde a primeira versão do script.

Como reproduzir o fato:

Após a página ser carregada,

  • clicar em qualquer local para que o navegador altere o elemento em foco;
  • em seguida tentar realizar a operação em questão.

OBS: caso o módulo de mensagem instantânea esteja habilitado e nas preferências do módulo esteja para carregar automaticamente o foco é alterado no momento em que a lista de contato é carregada.

Outras formas de reproduzir o fato:

  • após a página ser carregada escolher a opção "Nova Mensagem"
  • após a página ser carregada escolher a opção "Ferramentas - Preferências"
  • após a página ser carregada escolher a opção "Ferramentas - Editar Filtros"
  • após a página ser carregada escolher a opção "Ferramentas - Editar Pastas"
  • etc...

Levando em consideração a maneira como o script foi implementado (principalmente a forma de captura dos eventos no navegador, ou seja, o elemento utilizado como base é o document) realizar uma simples alteração é extremamente complexa.

Com isso está sendo estudado uma forma de contornar a situação ou mesmo reestruturar o script para que o mesmo atenda os propósitos aos quais o mesmo se destina.

comment:7 Changed 12 years ago by jakjr

O problema está corrigido. Já alterei o servidor de produção.

Foi testado no IE6, IE7, FF (~iceweasel) 2.0.17, 2.0.15, 2.0.9, 2.0b e 3.0.3.

O problema era com o tipo de evento. No firefox as repetições de teclas (quando você aperta e segura uma tecla) ocorre com o evento keypress. No IE com o evento keydown.

Sobre a maneira como o script foi implementado: Não sou desenvolvedor, apesar de ter criado boa parte do expressoMail. Estou utilizando o script shortcut.js ( http://www.openjs.com/scripts/events/keyboard_shortcuts/) no melhor estilo softwareLivre: RE-UTILIZANDO CÓDIGOS.

Muitos desenvolvedores adoram recriar ou reestruturas códigos existentes sem antes dar uma boa olhada neles. Talvez seja porque não entenderam o código e se não entendem o código qualquer alteração fica muito complexa.

Sobre isto:

10/16/08 10:47:43 alterado por rodsouza

Após verificar o histórico de alterações, consta que:

  • todas as alterações foram feitas pelo mesmo desenvolvedor; * nenhuma das alterações interferem >diretamente na funcionalidade em questão;

Assim sendo "a priori" o fato relatado possui outra causa, ou ainda é específico a um contexto, >não sendo causado por alterações.

Não sei se você trabalha com o eclipse, mas se trabalhar de uma olhada no Resource History do arquivo shortcut.js, principalmente em relação ao bug do shift+9 e shift+7.

Revisions 1356, 1431, 1453 e finalmente 1981 você verá o que aconteceu.

Bom, depois de tudo isto este é meu último Trac e meu último commit no projeto expressoMail.

Boa sorte a todos.

Att João Alfredo

comment:8 Changed 12 years ago by amuller

  • Owner changed from alguem to jakjr

comment:9 Changed 12 years ago by amuller

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

revisões 1981 e 1982 do repos. interno

comment:10 Changed 12 years ago by niltonneto

Publicado no r. 536

comment:11 Changed 9 years ago by niltonneto

  • Milestone Expresso Mail 1.232 deleted

Milestone Expresso Mail 1.232 deleted

Note: See TracTickets for help on using tickets.