FIX remove placeholder search text on readonly field

This commit is contained in:
Andrew Paxley 2024-10-08 17:04:07 +13:00
parent 483e944601
commit b5203fdc50
2 changed files with 5 additions and 1 deletions

View File

@ -139,7 +139,6 @@ trait SearchableDropdownTrait
return $emptyString; return $emptyString;
} }
} }
$name = $this->getName();
if ($this->getUseDynamicPlaceholder()) { if ($this->getUseDynamicPlaceholder()) {
if ($this->getIsSearchable()) { if ($this->getIsSearchable()) {
if (!$this->getIsLazyLoaded()) { if (!$this->getIsLazyLoaded()) {
@ -598,6 +597,9 @@ trait SearchableDropdownTrait
$field->setSource($this->sourceList); $field->setSource($this->sourceList);
$field->setReadonly(true); $field->setReadonly(true);
// Remove the text "Type to search..." on a read-only field
$field->setPlaceholder('');
return $field; return $field;
} }
} }

View File

@ -73,6 +73,8 @@ class SearchableDropdownTraitTest extends SapphireTest
$this->assertSame('My empty string', $field->getPlaceholder()); $this->assertSame('My empty string', $field->getPlaceholder());
$field->setPlaceholder('My placeholder'); $field->setPlaceholder('My placeholder');
$this->assertSame('My placeholder', $field->getPlaceholder()); $this->assertSame('My placeholder', $field->getPlaceholder());
$readonlyField = $field->performReadonlyTransformation();
$this->assertSame('', $readonlyField->getPlaceholder());
} }
public function testSeachContext(): void public function testSeachContext(): void