Melhoria da Usabilidade na Seleção de Grupos: Atualização da Interface para Seleção Intuitiva e Eficiente #5
+180
−20
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…o outro que haviam bugado por estarem datados com o passado
…amente dentro dos itens selecionados
moonshinerd
changed the title
moonshinerd
changed the title
moonshinerd
changed the title
moonshinerd
changed the title
moonshinerd
changed the title
sesostris
reviewed
Oct 8, 2024
There was a problem hiding this comment.
O que esse template tem a ver com o DualListBox????
There was a problem hiding this comment.
O que esse template tem a ver com o DualListBox????
There was a problem hiding this comment.
O que esse template tem a ver com o DualListBox????
There was a problem hiding this comment.
O que esse template tem a ver com o DualListBox????
There was a problem hiding this comment.
O que esse template tem a ver com o DualListBox????
| </div> | ||
|
|
||
| <!-- Duallistbox Visível --> | ||
| <select multiple="multiple" name="duallistbox_demo1[]" id="duallistbox_demo1" class="duallistbox"> |
There was a problem hiding this comment.
Tá certo isso? O name e id do select estão hardcoded. Se num form tiver dois ou mais campos com esse widget, todos terão o mesmo id/nome, que certamente trará problemas.
| <link rel="stylesheet" href="{% static 'css/related_widget_wrapper.css' %}"> | ||
| <div class="related-widget-wrapper" style="height: 100%;" {% if not model_has_limit_choices_to %}data-model-ref="{{ model_name }}"{% endif %}> | ||
| <!-- Renderiza o widget original, mas oculto --> | ||
| <div id="original-widget" style="display: none;"> |
There was a problem hiding this comment.
id hardcoded é errado. Cada elemento no HTML final tem que ter um id único
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Descrição do Pull Request
Este PR resolver a issue #4
Neste Pull Request, foram realizadas duas principais modificações. Primeiro, instalei a Django Debug Toolbar para facilitar o processo de depuração durante a configuração dos templates. A ferramenta foi adicionada no
INSTALLED_APPS, e verifiquei que oDEBUGestá configurado comoTrueno ambiente de desenvolvimento.Em segundo lugar, realizei uma customização nos widgets de seleção de grupos na interface do usuário. Anteriormente, a seleção de múltiplos grupos exigia o uso da tecla
Ctrl, o que não era uma abordagem intuitiva para os usuários. Para resolver isso, modifiquei a interface para permitir a seleção de múltiplos grupos de forma mais amigável, semelhante ao comportamento de "drag-and-drop" ou seleção em massa, sem a necessidade de pressionar oCtrl.Exemplos Visuais
Como era antes:
Referência de como deveria ser:
Como ficou:
Essas mudanças visam melhorar a usabilidade do sistema, permitindo que os usuários selecionem diversos grupos de maneira mais intuitiva e eficiente.
Além disso, justifico a inclusão do script diretamente no arquivo
related_widget_wrapper.htmlneste Pull Request para garantir que o comportamento do DualListBox esteja sempre sincronizado com o estado do widget original. Essa abordagem elimina a necessidade de múltiplas chamadas JavaScript externas e centraliza a lógica de manipulação do DOM (Document Object Model) dentro do próprio widget. Como resultado, melhoramos a eficiência e a manutenção do código, assegurando que as alterações nos elementos do DOM sejam refletidas em tempo real, e reduzimos a chance de erros relacionados à sincronização.