Problem/Motivation

Currently there is no way to hide Target field in the ALA widget, where other fields can be hidden.

Proposed resolution

Add another option "Enable target" in the widget settings

User interface changes

CommentFileSizeAuthor
#8 ala_hide-target-field-3205566_8.patch2.7 KBthe.tai.pen@gmail.com
#6 ala_hide-target-field-3205566_3.patch2.7 KBthe.tai.pen@gmail.com
#2 ala_hide-target-field-3205566_2.patch2.71 KBthe.tai.pen@gmail.com
ala-enable-target.png107.54 KBbotanic_spark

Comments

botanic_spark created an issue. See original summary.

the.tai.pen@gmail.com’s picture

the.tai.pen@gmail.com commented
StatusFileSize
new ala_hide-target-field-3205566_2.patch2.71 KB

Providing a patch to add the option "Enable target" in the widget settings. The option will be enabled by default to mimic the current behavior so the existing functionality isn't altered.

the.tai.pen@gmail.com’s picture

the.tai.pen@gmail.com commented
Status: Active » Needs review
botanic_spark’s picture

botanic_spark commented 
+++ b/src/Plugin/Field/FieldWidget/AdvanceLinkAttributesFieldWidget.php	(date 1616675490524)
@@ -62,6 +62,7 @@
+        'ala_enable_target' => 1,

Let's use the same pattern as for other settings: `ala_link_target`

botanic_spark’s picture

botanic_spark commented
Status: Needs review » Needs work
the.tai.pen@gmail.com’s picture

the.tai.pen@gmail.com commented
StatusFileSize
new ala_hide-target-field-3205566_3.patch2.7 KB

Sure. Sound like a good improvement

the.tai.pen@gmail.com’s picture

the.tai.pen@gmail.com commented
the.tai.pen@gmail.com’s picture

the.tai.pen@gmail.com commented
StatusFileSize
new ala_hide-target-field-3205566_8.patch2.7 KB

Adding updated patch

the.tai.pen@gmail.com’s picture

the.tai.pen@gmail.com commented
Status: Needs work » Needs review
kushan4157’s picture

kushan4157 commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.