BacLuc / ecamp3 / 28466554920

36%

devel: 36%

push

github

fix(program): avoid eager VOverlay mount causing Safari WeakMap error

Safari 17.1 throws "WeakMap keys must be objects or non-registered
symbols" in Vue's reactivity system when many VOverlay instances are
created simultaneously. This was triggered on the program/period view
after creating a new activity: the schedule-entries reload caused all
PicassoEntry components to mount, each eagerly mounting a VDialog
(DialogActivityEdit) via DialogForm's hardcoded `eager` prop. With
50+ activities on screen, this created 50+ reactive Sets at once inside
Vuetify's useStack(), hitting the Safari WeakMap limit.

Removing `eager` from DialogForm makes VOverlay mount lazily (only when
the dialog is actually opened), eliminating the simultaneous burst.
DetailPane.vue already passes `eager` explicitly via $attrs, so its
behaviour is unchanged.

Also remove the stale `:multiple="null"` attribute on VMenu in
SelectFilter — VMenu has no `multiple` prop, so this was always a
no-op.

fixes https://github.com/ecamp/ecamp3/issues/10185

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

840 of 4747 branches covered (17.7%)

Branch coverage included in aggregate %.

0 of 1 new or added line in 1 file covered. (0.0%)

4852 of 11177 relevant lines covered (43.41%)

984.17 hits per line