From 525c054cbf85daf5b2061c9a7b60e8cc893d2533 Mon Sep 17 00:00:00 2001
From: Lorenzo Perone <lorenzo.perone@yellowspace.net>
Date: Thu, 11 Jun 2026 15:19:27 +0000
Subject: [PATCH 1/3] Member modal: only run the write-back when the member
 dialog was open

The hide.bs.modal handler is bound to the shared #page-modal, so it ran
for every dialog (export, delete, values-show, ...). For those dialogs
attribute_values('destination',...) returns an empty list, which makes
modal_update() execute its 'all members removed' path - blanking the
first member input and removing the remaining member rows from the form.

Bail out unless the closing modal actually contains the member dialog.

Fixes #465
---
 resources/views/components/attribute/widget/options.blade.php | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/resources/views/components/attribute/widget/options.blade.php b/resources/views/components/attribute/widget/options.blade.php
index 5729aecd..dffba671 100644
--- a/resources/views/components/attribute/widget/options.blade.php
+++ b/resources/views/components/attribute/widget/options.blade.php
@@ -44,6 +44,10 @@
 							});
 
 							$('#page-modal').on('hide.bs.modal',function() {
+								// Only handle the close if the member-manage modal was the one open
+								if (! $(this).find('select#destination').length)
+									return;
+
 								var updated = modal_update(modal_attr,attribute_values('destination','select','option'));
 
 								if (updated.length)

From fb717b9794814ddb60950ac1a708732c6f3e4e9e Mon Sep 17 00:00:00 2001
From: Lorenzo Perone <lorenzo.perone@yellowspace.net>
Date: Thu, 11 Jun 2026 15:19:27 +0000
Subject: [PATCH 2/3] Member modal: make the filter work in WebKit browsers

The filter hid <option> elements with jQuery .toggle(), which WebKit
(Safari, Chrome) does not support - typing in 'Filter Members' had no
effect there.

Detach non-matching options instead, restoring them on the next filter
run, and keep the visible list sorted. Only the Available Members list
is filtered, so a filtered view at close time can never drop current
members from the write-back.

Fixes #466
---
 .../views/modals/member-manage.blade.php      | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/resources/views/modals/member-manage.blade.php b/resources/views/modals/member-manage.blade.php
index 4e5bcbe3..e70201d3 100644
--- a/resources/views/modals/member-manage.blade.php
+++ b/resources/views/modals/member-manage.blade.php
@@ -114,16 +114,19 @@
 	});
 
 	var filter = _.debounce(function(filter) {
-		$('select#source option').each(function() {
-			var option = $(this).text().toLowerCase();
+		var select = $('select#source');
 
-			$(this).toggle(option.indexOf(filter) > -1);
-		});
+		// Restore options hidden by a previous filter run
+		select.append(select.data('filtered') || $());
 
-		$('select#destination option').each(function() {
-			var option = $(this).text().toLowerCase();
+		// Detach non-matching options - hiding <option> with CSS is not supported in all browsers
+		select.data('filtered',select.find('option').filter(function() {
+			return $(this).text().toLowerCase().indexOf(filter) === -1;
+		}).detach());
 
-			$(this).toggle(option.indexOf(filter) > -1);
-		});
+		// Keep the visible list sorted
+		select.append(select.find('option').detach().sort(function(a,b) {
+			return $(a).text().localeCompare($(b).text());
+		}));
 	}, 500);
 </script>
\ No newline at end of file

From 7a2d45ca48fecd7849e2abda5da989e1e16062a3 Mon Sep 17 00:00:00 2001
From: Lorenzo Perone <lorenzo.perone@yellowspace.net>
Date: Thu, 11 Jun 2026 15:19:27 +0000
Subject: [PATCH 3/3] Exclude hidden overflow values of read-only renders from
 member processing

Values beyond pla.limit.values are rendered as hidden inputs, but
without the conditional no-edit class that visible rows get. When a
template renders the member attribute on the read-only tab of an entry,
the member dialog and attribute_values() (which exclude .no-edit to
skip read-only renders) read those hidden inputs twice. The duplicate
entries make removals impossible: removing one duplicate from the
dialog leaves the other, so modal_update() keeps the member.

Apply the same no-edit condition to the hidden inputs.

Fixes #467
---
 resources/views/components/attribute/values.blade.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/resources/views/components/attribute/values.blade.php b/resources/views/components/attribute/values.blade.php
index 72039be8..d5c99dd9 100644
--- a/resources/views/components/attribute/values.blade.php
+++ b/resources/views/components/attribute/values.blade.php
@@ -30,7 +30,7 @@
 		@endforeach
 
 		@foreach(($x=$values->skip(config('pla.limit.values'))->filter()) as $key => $value)
-			<input type="text" class="d-none" name="{{ $o->name_lc }}[{{ $langtag }}][]" value="{{ $value }}">
+			<input type="text" @class(['d-none','no-edit'=>(! ($editable ?? FALSE) && ($o->dn))]) name="{{ $o->name_lc }}[{{ $langtag }}][]" value="{{ $value }}">
 		@endforeach
 
 		@if($x->count())