HOTFIX: OrderCatalog scroll To Item #1026

Merged
jsegarra merged 9 commits from hotfix_scrollToItem into master 2024-12-09 08:44:08 +00:00
Member

Después de añadir un producto, se refresca y pierde el scroll

Después de añadir un producto, se refresca y pierde el scroll
jsegarra added 2 commits 2024-11-30 00:35:55 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
0f0d906e08
fix: #6896 OrderCatalogItem actions
jsegarra requested review from alexm 2024-11-30 00:36:26 +00:00
jsegarra requested review from jon 2024-11-30 00:36:26 +00:00
jsegarra added 1 commit 2024-11-30 00:38:15 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
6622135271
perf: #6896 change path import
jsegarra changed title from fix: hotfix_scrollToItem to HOTFIX: OrderCatalog scroll To Item 2024-12-02 08:20:38 +00:00
alexm requested changes 2024-12-02 11:58:13 +00:00
Dismissed
@ -79,0 +83,4 @@
const scrollToItem = async (id) => {
await nextTick();
setTimeout(() => {
Member

Y si lo tuviese que cargar tardase 101ms ya no iria?

Y si lo tuviese que cargar tardase 101ms ya no iria?
Author
Member

Le puse el nextTick para que el DOM terminase de montarse pero no terminaba de funcionar
Con el setTimeout conseguí el efecto deseado
Iría de todas maneras porque el timeout no depende de la petición anterior

Le puse el nextTick para que el DOM terminase de montarse pero no terminaba de funcionar Con el setTimeout conseguí el efecto deseado Iría de todas maneras porque el timeout no depende de la petición anterior
Member
https://chatgpt.com/c/675000bd-53e8-8011-8a7a-0481f274180d
Author
Member

Resuelto con requestAnimationFrame
He usado un console.error y solo se muestra 1 vez

Resuelto con requestAnimationFrame He usado un console.error y solo se muestra 1 vez
jsegarra requested review from alexm 2024-12-02 12:20:12 +00:00
jsegarra added 1 commit 2024-12-02 12:20:18 +00:00
gitea/salix-front/pipeline/pr-master Build queued... Details
f3834e92d1
Merge branch 'master' into hotfix_scrollToItem
jsegarra added 1 commit 2024-12-02 12:34:59 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
624764ba1d
Merge branch 'master' into hotfix_scrollToItem
jsegarra added 1 commit 2024-12-03 14:33:20 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
dcbb0da8c0
Merge branch 'master' into hotfix_scrollToItem
jsegarra added 1 commit 2024-12-04 08:15:06 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
98b8124577
perf: remove setTimeOut insteaduse requestAnimationFrame
jsegarra added 1 commit 2024-12-09 07:45:20 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
edef54dae1
Merge branch 'master' into hotfix_scrollToItem
alexm approved these changes 2024-12-09 08:27:20 +00:00
jsegarra added 1 commit 2024-12-09 08:42:48 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
f049337f95
Merge branch 'master' into hotfix_scrollToItem
jon approved these changes 2024-12-09 08:43:40 +00:00
jsegarra merged commit ce72388d5e into master 2024-12-09 08:44:08 +00:00
jsegarra deleted branch hotfix_scrollToItem 2024-12-09 08:44:08 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix-front#1026
No description provided.