Skip to content

Replace the column resizing process with one based on drag & drop events [PREVIEW]#368

Open
nicoe wants to merge 1587 commits intocoopengo:masterfrom
nicoe:PCLAS-1066/jira_col_resizing
Open

Replace the column resizing process with one based on drag & drop events [PREVIEW]#368
nicoe wants to merge 1587 commits intocoopengo:masterfrom
nicoe:PCLAS-1066/jira_col_resizing

Conversation

@nicoe
Copy link
Collaborator

@nicoe nicoe commented Dec 10, 2024

Fix PCLAS-1066

JCavallo and others added 30 commits September 2, 2024 15:51
This is hacky, but allows to have the same behaviour than
that of gtk / proteus

Fix PCLAS-713
Fix PCLAS-597

Co-authored-by: Jean Cavallo <jean.cavallo@hotmail.fr>
…W] (coopengo#187)

For performance reasons it's better not to mix DOM reading and
modifying especially in JS loops.

PCLAS-772
…REVIEW] (coopengo#50)

Co-authored-by: Jean Cavallo <jean.cavallo@hotmail.fr>
* Optimize instance creation in __getattr__ [PREVIEW]

Accessing a M2O field from a X2M field could result in as many
instantiations as there are records in the X2M field. It could be a
performance bottleneck due to the many creations of Model kwargs and
potential useless calls of transaction.set_context().

Thus the optimization is twofold:

 - Create an additional code path when the M2O field modifies the
   context
 - Cache the kwargs creation process to prevent extra calls resulting in
   the same dictionary

* Use read to fetch reverse field value of O2M field in get [PREVIEW]

Using a browse there in order to get the id is a performance bottleneck
as it will result in the instanciation of potentially thousands of
models.

---------

Co-authored-by: Jean Cavallo <jean.cavallo@hotmail.fr>
Closes PCLAS-818

---------

Co-authored-by: Jean Cavallo <jean.cavallo@hotmail.fr>
nicoe and others added 18 commits January 23, 2025 15:51
…opengo#384)

jQuery's .text() does an .empty() call unconditionally which result in
the browser thinking the DOM has changed even when the textual is
identical
…REVIEW] (coopengo#372)

Fix PCLAS-1280
https://bugs.tryton.org/13763
---------

Co-authored-by: Jean Cavallo <jean.cavallo@hotmail.fr>
@JCavallo
Copy link
Contributor

JCavallo commented Mar 7, 2025

Fait quelques tests :

  • La version précédente est plus fluide. Notamment, cette version a un délai (~300ms au feeling) entre le début du drag-n-drop et le moment où le déplacement a lieu visuellement)
  • On a le problème (corrigé via un algorighme un peu bancal sur la version précédente) où la colonne qui est resize ne correspond pas à celle dont on a pris le handler

@nicoe
Copy link
Collaborator Author

nicoe commented Mar 11, 2025

Sur le délai, j'ai essayé plusieurs techniques mais je l'ai toujours. Je pense que c'est inhérent au fait d'utiliser le drag & drop car dans mes tests, JIRA présente lui aussi ce délai initial.

Sur le problème de la colonne qui n'est pas celle redimensionnée, je ne comprends pas : les colonnes précédentes sont fixées et je ne l'observe pas.

nicoe and others added 7 commits March 13, 2025 11:28
This one is a complicated beast:

    - having defined an `overflow-x: auto` in CSS on modal-body make it
      a new Block Formatting Context
    - Block Formatting Context contains their floating elements
    - This is why although the ul menu is in the same stacking context
      than the modal footer, the menu does not appear above it.
    - Thus setting the `overflow` property will also reset the
      `overflow-x` property and since the value is `visible` it does not
      create a new Block Formatting Context, ensuring the `z-index` take
      effect.

Fix PCLAS-1506
* Unconditionally enable button after a clic [CUSTOM]

PCLAS-1522

* Only call the button state setting when the call to button failed

* Remove console.log
…n-coopengo-new into PCLAS-1066/jira_col_resizing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants