Skip to content

Conversation

@havaguernicoopengo
Copy link
Contributor

Fix PCLAS-1531

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
…W] (coopengo#187)

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

PCLAS-772
* 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 <[email protected]>
Closes PCLAS-818

---------

Co-authored-by: Jean Cavallo <[email protected]>
…pengo#219)

* Optimize instance creation in ModelStorage.__getattr__ [PREVIEW]

https://bugs.tryton.org/13245

* Sort X2M fields according to the order [PREVIEW]

Fix PCLAS-889
https://bugs.tryton.org/13288
nicoe and others added 29 commits January 23, 2025 18:02
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
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.