From 43317ea2854f7b6cc84f97f420ce199c5949f5fb Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 21 Jul 2026 13:20:26 +0000
Subject: [PATCH 1/2] Rebuild sample 346 to focus table cells via native
 set_focus action

Replace the inline <script> block and the custom-JS follow-up actions with
the framework's cs_event-set_focus follow-up action and the native s_focus
focus report, so no JavaScript is shipped with the view. Each editable cell
carries a stable control id (<column>-<row>) that set_focus targets directly,
and NEXT reads the currently focused cell from client->get( )-s_focus-id.

As the sample no longer needs native JS, move it from src/00/05 to src/01/02
(framework actions), mark the DESCRIPT with the (A) capability marker, and
regenerate both overview catalogs.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RjiwGw55EN3XxRMqfKg3eM
---
 src/00/05/z2ui5_cl_demo_app_346.clas.abap     | 253 ------------------
 src/00/z2ui5_cl_sample_app_g01.clas.abap      |   1 -
 src/01/02/z2ui5_cl_demo_app_346.clas.abap     | 243 +++++++++++++++++
 .../02}/z2ui5_cl_demo_app_346.clas.xml        |   2 +-
 src/01/z2ui5_cl_demo_app_g00.clas.abap        |   3 +-
 5 files changed, 246 insertions(+), 256 deletions(-)
 delete mode 100644 src/00/05/z2ui5_cl_demo_app_346.clas.abap
 create mode 100644 src/01/02/z2ui5_cl_demo_app_346.clas.abap
 rename src/{00/05 => 01/02}/z2ui5_cl_demo_app_346.clas.xml (87%)

diff --git a/src/00/05/z2ui5_cl_demo_app_346.clas.abap b/src/00/05/z2ui5_cl_demo_app_346.clas.abap
deleted file mode 100644
index cc344f41f..000000000
--- a/src/00/05/z2ui5_cl_demo_app_346.clas.abap
+++ /dev/null
@@ -1,253 +0,0 @@
-CLASS z2ui5_cl_demo_app_346 DEFINITION PUBLIC.
-
-  PUBLIC SECTION.
-    INTERFACES z2ui5_if_app.
-
-    TYPES:
-      BEGIN OF ty_s_row,
-        index       TYPE i,
-        title       TYPE string,
-        value       TYPE string,
-        description TYPE string,
-        icon        TYPE string,
-        info        TYPE string,
-        checkbox    TYPE abap_bool,
-      END OF ty_s_row.
-
-    DATA t_tab TYPE STANDARD TABLE OF ty_s_row WITH EMPTY KEY.
-    DATA focuscolumn TYPE string.
-    DATA focusrow TYPE string.
-    DATA focusid TYPE string READ-ONLY.
-
-  PROTECTED SECTION.
-    CONSTANTS:
-      BEGIN OF c_id,
-        index       TYPE string VALUE `Index`,
-        title       TYPE string VALUE `Title`,
-        color       TYPE string VALUE `Color`,
-        info        TYPE string VALUE `Info`,
-        checkbox    TYPE string VALUE `Checkbox`,
-        description TYPE string VALUE `Description`,
-      END OF c_id.
-
-    DATA client TYPE REF TO z2ui5_if_client.
-
-    METHODS set_view.
-
-    METHODS next_focus.
-    METHODS focus.
-    METHODS default_focus.
-
-  PRIVATE SECTION.
-ENDCLASS.
-
-
-CLASS z2ui5_cl_demo_app_346 IMPLEMENTATION.
-
-  METHOD set_view.
-
-    DATA(view) = z2ui5_cl_xml_view=>factory( ).
-
-    view->_generic( name = `script`
-                    ns   = `html` )->_cc_plain_xml( `window.addEventListener('focus', function(e) {`
-                                                && `  try {`
-                                                && `    const focusCtrlId = sap.ui.getCore().getCurrentFocusedControlId(); `
-                                                && `    if (!focusCtrlId) {`
-                                                && `      return;`
-                                                && `    }`
-                                                && `    const customData = sap.ui.core.Element.getElementById(focusCtrlId).getCustomData()[0];`
-                                                && `    if (!customData) {`
-                                                && `      return;`
-                                                && `    }`
-                                                && `    const column = customData.getProperty("value");`
-                                                && `    if (!column) {`
-                                                && `      return;`
-                                                && `    }`
-                                                && `    const m = focusCtrlId.match(/(\d+$)/);`
-                                                && `    const model = z2ui5.oView.getModel() ;`
-                                                && `    model.setProperty("/FOCUSID",focusCtrlId);`
-                                                && `    model.setProperty("/FOCUSCOLUMN",column);`
-                                                && `    model.setProperty("/FOCUSROW",m[1]);`
-                                                && `  } catch(e){}`
-                                                && `}, true);`
-                                                && ``
-                                                && `z2ui5.determineFocusId = (column, row) => { `
-                                                && `  try {`
-                                                && `    const selector = "td:has([data-columnid='" + column + "']) > div";`
-                                                && `    const id = document.querySelectorAll(selector)[row].id;`
-                                                && `    z2ui5.oView.getModel().setProperty("/FOCUSID",id);`
-                                                && `    const element = sap.ui.core.Element.getElementById(id);`
-                                                && `    if (!element) {`
-                                                && `      return;`
-                                                && `    }`
-                                                && `    const focus = element.getFocusInfo();`
-                                                && `    element.applyFocusInfo(focus);`
-                                                && `  } catch(e){}`
-                                                && `}` ).
-
-    DATA(page) = view->shell(
-        )->page(
-            title          = `abap2UI5 - Tables and focus`
-            navbuttonpress = client->_event( `BACK` )
-            shownavbutton  = abap_true ).
-
-    DATA(tab) = page->table(
-            client->_bind_edit( t_tab )
-        )->header_toolbar(
-            )->overflow_toolbar(
-                )->label( `Column Id` )->input( submit      = client->_event( `FOCUS` )
-                                                value       = client->_bind_edit( focuscolumn )
-                                                placeholder = `Focus Column`
-                                                width       = `10%`
-                )->label( `Row Index` )->input( submit      = client->_event( `FOCUS` )
-                                                value       = client->_bind_edit( focusrow )
-                                                placeholder = `Focus Row`
-                                                width       = `10%`
-                                                type        = `Number`
-                )->button(
-                    text  = `Set Focus`
-                    press = client->_event( `FOCUS` )
-                )->button(
-                    text  = `Next Focus`
-                    press = client->_event( `ENTER` )
-                )->button(
-                    text  = `Reset Focus`
-                    press = client->_event( `RESET` )
-                )->title( client->_bind( focusid )
-                )->toolbar_spacer(
-        )->get_parent( )->get_parent( ).
-
-    tab->columns(
-        )->column(
-            )->text( `Index` )->get_parent(
-        )->column(
-            )->text( `Title` )->get_parent(
-        )->column(
-            )->text( `Color` )->get_parent(
-        )->column(
-            )->text( `Info` )->get_parent(
-        )->column(
-            )->text( `Checkbox` )->get_parent(
-        )->column(
-            )->text( `Description` ).
-
-    tab->items( )->column_list_item( selected = `{SELKZ}`
-      )->cells(
-          )->text( `{INDEX}`
-          )->input( value  = `{TITLE}`
-                    submit = client->_event( `ENTER` )
-          )->get( )->custom_data( )->core_custom_data(
-                     key        = `ColumnId`
-                     value      = c_id-title
-                     writetodom = abap_true
-          )->get_parent( )->get_parent(
-          )->input( value  = `{VALUE}`
-                    submit = client->_event( `ENTER` )
-          )->get( )->custom_data( )->core_custom_data(
-                     key        = `ColumnId`
-                     value      = c_id-color
-                     writetodom = abap_true
-          )->get_parent( )->get_parent(
-          )->input( value  = `{INFO}`
-                    submit = client->_event( `ENTER` )
-          )->get( )->custom_data( )->core_custom_data(
-                     key        = `ColumnId`
-                     value      = c_id-info
-                     writetodom = abap_true
-          )->get_parent( )->get_parent(
-          )->checkbox( `{CHECKBOX}`
-          )->get( )->custom_data( )->core_custom_data(
-                     key        = `ColumnId`
-                     value      = c_id-checkbox
-                     writetodom = abap_true
-          )->get_parent( )->get_parent(
-          )->input( value  = `{DESCRIPTION}`
-                    submit = client->_event( `ENTER` )
-          )->get( )->custom_data( )->core_custom_data(
-                     key        = `ColumnId`
-                     value      = c_id-description
-                     writetodom = abap_true ).
-
-    client->view_display( view->stringify( ) ).
-    focus( ).
-
-  ENDMETHOD.
-
-
-  METHOD z2ui5_if_app~main.
-
-    me->client = client.
-
-    IF client->check_on_init( ).
-
-      t_tab = VALUE #(
-          ( index = 0 title = `entry 01`  value = `red`    info = `completed`  description = `this is a description` checkbox = abap_true )
-          ( index = 1 title = `entry 02`  value = `blue`   info = `completed`  description = `this is a description` checkbox = abap_true )
-          ( index = 2 title = `entry 03`  value = `green`  info = `completed`  description = `this is a description` checkbox = abap_true )
-          ( index = 3 title = `entry 04`  value = `orange` info = `completed`  description = `` checkbox = abap_true )
-          ( index = 4 title = `entry 05`  value = `grey`   info = `completed`  description = `this is a description` checkbox = abap_true )
-          ( index = 5 ) ).
-
-      default_focus( ).
-      set_view( ).
-      RETURN.
-
-    ENDIF.
-
-    CASE client->get( )-event.
-      WHEN `BACK`.
-        client->view_destroy( ).
-        client->nav_app_leave( ).
-      WHEN `FOCUS`.
-        focus( ).
-      WHEN `RESET`.
-        default_focus( ).
-        focus( ).
-      WHEN `ENTER`.
-        next_focus( ).
-        focus( ).
-    ENDCASE.
-
-    client->view_model_update( ).
-
-  ENDMETHOD.
-
-
-  METHOD next_focus.
-
-    focuscolumn = SWITCH #(
-                    focuscolumn
-                      WHEN c_id-title THEN c_id-color
-                      WHEN c_id-color THEN c_id-info
-                      WHEN c_id-info  THEN c_id-checkbox
-                      WHEN c_id-checkbox THEN c_id-description
-                      ELSE c_id-title ).
-
-    IF focuscolumn = c_id-title.
-
-      IF line_exists( t_tab[ focusrow + 2 ] ).
-        focusrow = condense( CONV i( focusrow + 1 ) ).
-
-      ELSE.
-        focusrow = `0`.
-      ENDIF.
-    ENDIF.
-
-  ENDMETHOD.
-
-
-  METHOD focus.
-
-    client->follow_up_action( `z2ui5.determineFocusId("` && focuscolumn && `", "` && focusrow && `")` ).
-
-  ENDMETHOD.
-
-
-  METHOD default_focus.
-
-    focuscolumn = `Title`.
-    focusrow = `0`.
-
-  ENDMETHOD.
-
-ENDCLASS.
diff --git a/src/00/z2ui5_cl_sample_app_g01.clas.abap b/src/00/z2ui5_cl_sample_app_g01.clas.abap
index f13278613..4ee76127c 100644
--- a/src/00/z2ui5_cl_sample_app_g01.clas.abap
+++ b/src/00/z2ui5_cl_sample_app_g01.clas.abap
@@ -275,7 +275,6 @@ CLASS z2ui5_cl_sample_app_g01 IMPLEMENTATION.
       ( group = `only with javascript and css and html` header = `Messages with Styles I` sub = `` app = `z2ui5_cl_demo_app_310` )
       ( group = `only with javascript and css and html` header = `Messages with Styles II` sub = `` app = `z2ui5_cl_demo_app_311` )
       ( group = `only with javascript and css and html` header = `PDF Viewer` sub = `Display PDFs via iframe` app = `z2ui5_cl_demo_app_079` )
-      ( group = `only with javascript and css and html` header = `tab` sub = `focus edit controls` app = `z2ui5_cl_demo_app_346` )
       ( group = `only with javascript and css and html` header = `Tile` sub = `KPI Tile` app = `z2ui5_cl_demo_app_277` )
       ( group = `only with javascript and css and html` header = `tree` sub = `drag & drop` app = `z2ui5_cl_demo_app_317` )
       ( group = `only with javascript and css and html` header = `tree` sub = `popup select - state` app = `z2ui5_cl_demo_app_178` )
diff --git a/src/01/02/z2ui5_cl_demo_app_346.clas.abap b/src/01/02/z2ui5_cl_demo_app_346.clas.abap
new file mode 100644
index 000000000..3c2c258df
--- /dev/null
+++ b/src/01/02/z2ui5_cl_demo_app_346.clas.abap
@@ -0,0 +1,243 @@
+CLASS z2ui5_cl_demo_app_346 DEFINITION PUBLIC.
+
+  PUBLIC SECTION.
+    INTERFACES z2ui5_if_app.
+
+    TYPES:
+      BEGIN OF ty_s_row,
+        index       TYPE i,
+        title       TYPE string,
+        value       TYPE string,
+        info        TYPE string,
+        checkbox    TYPE abap_bool,
+        description TYPE string,
+      END OF ty_s_row.
+    DATA t_tab TYPE STANDARD TABLE OF ty_s_row WITH EMPTY KEY.
+
+    DATA focuscolumn TYPE string.
+    DATA focusrow    TYPE string.
+    DATA focusid     TYPE string READ-ONLY.
+
+  PROTECTED SECTION.
+    CONSTANTS:
+      BEGIN OF cs_column,
+        title       TYPE string VALUE `Title`,
+        color       TYPE string VALUE `Color`,
+        info        TYPE string VALUE `Info`,
+        checkbox    TYPE string VALUE `Checkbox`,
+        description TYPE string VALUE `Description`,
+      END OF cs_column.
+
+    DATA client TYPE REF TO z2ui5_if_client.
+
+    METHODS on_init.
+    METHODS on_event.
+    METHODS view_display.
+    METHODS focus.
+    METHODS read_focus.
+    METHODS next_focus.
+    METHODS default_focus.
+
+  PRIVATE SECTION.
+ENDCLASS.
+
+
+CLASS z2ui5_cl_demo_app_346 IMPLEMENTATION.
+
+  METHOD z2ui5_if_app~main.
+
+    me->client = client.
+
+    IF client->check_on_init( ).
+      on_init( ).
+    ELSEIF client->check_on_navigated( ).
+      view_display( ).
+    ELSEIF client->check_on_event( ).
+      on_event( ).
+    ENDIF.
+
+  ENDMETHOD.
+
+
+  METHOD on_init.
+
+    t_tab = VALUE #(
+        ( index = 0 title = `entry 01` value = `red`    info = `completed` description = `this is a description` checkbox = abap_true )
+        ( index = 1 title = `entry 02` value = `blue`   info = `completed` description = `this is a description` checkbox = abap_true )
+        ( index = 2 title = `entry 03` value = `green`  info = `completed` description = `this is a description` checkbox = abap_true )
+        ( index = 3 title = `entry 04` value = `orange` info = `completed` description = ``                     checkbox = abap_true )
+        ( index = 4 title = `entry 05` value = `grey`   info = `completed` description = `this is a description` checkbox = abap_true )
+        ( index = 5 ) ).
+
+    default_focus( ).
+    view_display( ).
+    focus( ).
+
+  ENDMETHOD.
+
+
+  METHOD on_event.
+
+    CASE client->get( )-event.
+      WHEN `FOCUS`.
+        focus( ).
+      WHEN `NEXT`.
+        read_focus( ).
+        next_focus( ).
+        focus( ).
+      WHEN `RESET`.
+        default_focus( ).
+        focus( ).
+    ENDCASE.
+
+    client->view_model_update( ).
+
+  ENDMETHOD.
+
+
+  METHOD view_display.
+
+    DATA(view) = z2ui5_cl_xml_view=>factory( ).
+
+    DATA(page) = view->shell(
+        )->page(
+            title          = `abap2UI5 - Focus a Table Cell`
+            navbuttonpress = client->_event_nav_app_leave( )
+            shownavbutton  = client->check_app_prev_stack( ) ).
+
+    page->message_strip(
+        text     = `Set the keyboard focus to any editable table cell from the backend - type a column id ` &&
+                   `(Title, Color, Info, Checkbox or Description) and a row index, then press Set Focus, or ` &&
+                   `use Next / Reset. No JavaScript is shipped with the view: the set_focus follow-up action ` &&
+                   `targets the cell by its control id, and the framework reports the focused cell back in s_focus.`
+        type     = `Information`
+        showicon = abap_true
+        class    = `sapUiSmallMargin` ).
+
+    DATA(tab) = page->table(
+            client->_bind_edit( t_tab )
+        )->header_toolbar(
+            )->overflow_toolbar(
+                )->title( client->_bind( focusid )
+                )->toolbar_spacer(
+                )->label( `Column Id`
+                )->input(
+                    value       = client->_bind_edit( focuscolumn )
+                    submit      = client->_event( `FOCUS` )
+                    placeholder = `Column`
+                    width       = `8rem`
+                )->label( `Row Index`
+                )->input(
+                    value       = client->_bind_edit( focusrow )
+                    submit      = client->_event( `FOCUS` )
+                    placeholder = `Row`
+                    type        = `Number`
+                    width       = `6rem`
+                )->button(
+                    text  = `Set Focus`
+                    press = client->_event( `FOCUS` )
+                )->button(
+                    text  = `Next Focus`
+                    press = client->_event( `NEXT` )
+                )->button(
+                    text  = `Reset Focus`
+                    press = client->_event( `RESET` )
+        )->get_parent( )->get_parent( ).
+
+    tab->columns(
+        )->column( )->text( `Index` )->get_parent(
+        )->column( )->text( `Title` )->get_parent(
+        )->column( )->text( `Color` )->get_parent(
+        )->column( )->text( `Info` )->get_parent(
+        )->column( )->text( `Checkbox` )->get_parent(
+        )->column( )->text( `Description` ).
+
+    tab->items(
+        )->column_list_item(
+            )->cells(
+                )->text( `{INDEX}`
+                )->input(
+                    id     = cs_column-title
+                    value  = `{TITLE}`
+                    submit = client->_event( `NEXT` )
+                )->input(
+                    id     = cs_column-color
+                    value  = `{VALUE}`
+                    submit = client->_event( `NEXT` )
+                )->input(
+                    id     = cs_column-info
+                    value  = `{INFO}`
+                    submit = client->_event( `NEXT` )
+                )->checkbox(
+                    id       = cs_column-checkbox
+                    selected = `{CHECKBOX}`
+                )->input(
+                    id     = cs_column-description
+                    value  = `{DESCRIPTION}`
+                    submit = client->_event( `NEXT` ) ).
+
+    client->view_display( view->stringify( ) ).
+
+  ENDMETHOD.
+
+
+  METHOD focus.
+
+    focusid = |{ focuscolumn }-{ focusrow }|.
+
+    client->follow_up_action(
+        val   = z2ui5_if_client=>cs_event-set_focus
+        t_arg = VALUE #( ( focusid ) ) ).
+
+  ENDMETHOD.
+
+
+  METHOD read_focus.
+
+    SPLIT client->get( )-s_focus-id AT `-` INTO DATA(col) DATA(row).
+
+    IF row IS NOT INITIAL
+        AND row CO `0123456789`
+        AND ( col = cs_column-title
+           OR col = cs_column-color
+           OR col = cs_column-info
+           OR col = cs_column-checkbox
+           OR col = cs_column-description ).
+
+      focuscolumn = col.
+      focusrow    = row.
+    ENDIF.
+
+  ENDMETHOD.
+
+
+  METHOD next_focus.
+
+    focuscolumn = SWITCH #( focuscolumn
+                            WHEN cs_column-title    THEN cs_column-color
+                            WHEN cs_column-color    THEN cs_column-info
+                            WHEN cs_column-info     THEN cs_column-checkbox
+                            WHEN cs_column-checkbox THEN cs_column-description
+                            ELSE cs_column-title ).
+
+    IF focuscolumn = cs_column-title.
+
+      DATA(nextrow) = CONV i( focusrow ) + 1.
+      IF line_exists( t_tab[ nextrow + 1 ] ).
+        focusrow = |{ nextrow }|.
+      ELSE.
+        focusrow = `0`.
+      ENDIF.
+    ENDIF.
+
+  ENDMETHOD.
+
+
+  METHOD default_focus.
+
+    focuscolumn = cs_column-title.
+    focusrow    = `0`.
+
+  ENDMETHOD.
+
+ENDCLASS.
diff --git a/src/00/05/z2ui5_cl_demo_app_346.clas.xml b/src/01/02/z2ui5_cl_demo_app_346.clas.xml
similarity index 87%
rename from src/00/05/z2ui5_cl_demo_app_346.clas.xml
rename to src/01/02/z2ui5_cl_demo_app_346.clas.xml
index cf9a24b83..0fe89bc6f 100644
--- a/src/00/05/z2ui5_cl_demo_app_346.clas.xml
+++ b/src/01/02/z2ui5_cl_demo_app_346.clas.xml
@@ -5,7 +5,7 @@
    <VSEOCLASS>
     <CLSNAME>Z2UI5_CL_DEMO_APP_346</CLSNAME>
     <LANGU>E</LANGU>
-    <DESCRIPT>tab - focus edit controls</DESCRIPT>
+    <DESCRIPT>Focus - Set Focus in a Table Cell (A)</DESCRIPT>
     <STATE>1</STATE>
     <CLSCCINCL>X</CLSCCINCL>
     <FIXPT>X</FIXPT>
diff --git a/src/01/z2ui5_cl_demo_app_g00.clas.abap b/src/01/z2ui5_cl_demo_app_g00.clas.abap
index 69b4c274e..a53091a13 100644
--- a/src/01/z2ui5_cl_demo_app_g00.clas.abap
+++ b/src/01/z2ui5_cl_demo_app_g00.clas.abap
@@ -218,6 +218,7 @@ CLASS z2ui5_cl_demo_app_g00 IMPLEMENTATION.
       ( group = `Basic I` header = `Event` sub = `Handle events & change the view` app = `z2ui5_cl_demo_app_004` )
       ( group = `Basic I` header = `Formatter` sub = `Date Object for DatePicker` app = `z2ui5_cl_demo_app_457` )
       ( group = `Basic I` header = `Formatter` sub = `Date Objects for PlanningCalendar` app = `z2ui5_cl_demo_app_456` )
+      ( group = `Basic I` header = `Formatter` sub = `Inline Icons` app = `z2ui5_cl_demo_app_466` )
       ( group = `Basic I` header = `Formatter` sub = `Simple` app = `z2ui5_cl_demo_app_453` )
       ( group = `Basic I` header = `Formatter` sub = `Use via core:require` app = `z2ui5_cl_demo_app_450` )
       ( group = `Basic I` header = `Message` sub = `Backend Processing` app = `z2ui5_cl_demo_app_008` )
@@ -250,6 +251,7 @@ CLASS z2ui5_cl_demo_app_g00 IMPLEMENTATION.
       ( group = `Basic II` header = `Browser` sub = `Open Telephon, Email usw (A)` app = `z2ui5_cl_demo_app_316` )
       ( group = `Basic II` header = `Browser` sub = `Title (A)` app = `z2ui5_cl_demo_app_125` )
       ( group = `Basic II` header = `Focus` sub = `Jump with the focus (A)` app = `z2ui5_cl_demo_app_189` )
+      ( group = `Basic II` header = `Focus` sub = `Set Focus in a Table Cell (A)` app = `z2ui5_cl_demo_app_346` )
       ( group = `Basic II` header = `Focus` sub = `Set Focus in Textfield (A)` app = `z2ui5_cl_demo_app_133` )
       ( group = `Basic II` header = `Input` sub = `Clipboard (A)` app = `z2ui5_cl_demo_app_325` )
       ( group = `Basic II` header = `Input` sub = `Hide/show Soft Keyboard (A)` app = `z2ui5_cl_demo_app_352` )
@@ -261,7 +263,6 @@ CLASS z2ui5_cl_demo_app_g00 IMPLEMENTATION.
       ( group = `Basic II` header = `More` sub = `File Uploader (C)` app = `z2ui5_cl_demo_app_074` )
       ( group = `Basic II` header = `More` sub = `Geoloaction (C)` app = `z2ui5_cl_demo_app_120` )
       ( group = `Basic II` header = `More` sub = `Message Manager (C)` app = `z2ui5_cl_demo_app_467` )
-      ( group = `Basic II` header = `More` sub = `MessageStrip Inline Icons` app = `z2ui5_cl_demo_app_466` )
       ( group = `Basic II` header = `More` sub = `Multi Input (C)` app = `z2ui5_cl_demo_app_078` )
       ( group = `Basic II` header = `More` sub = `Panel, setExpanded (A)` app = `z2ui5_cl_demo_app_448` )
       ( group = `Basic II` header = `More` sub = `PDF Viewer Display (A)` app = `z2ui5_cl_demo_app_449` )

From a5fb6dd7c9bd9f127272d70ef1042e2c633c303d Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Tue, 21 Jul 2026 13:42:28 +0000
Subject: [PATCH 2/2] Fix set_focus by giving table cells stable declared ids

The first rewrite bound the table via items="{...}" and targeted cells by a
constructed id like Color-2. That never worked: with extended change detection
(the default for sap.m.Table) UI5 clones the item template under randomly
generated ids (uid('clone')), so view.byId - and therefore SET_FOCUS - cannot
resolve a constructed id. That is why the original sample discovered the id via
a DOM query in JavaScript.

Build the rows explicitly instead (no aggregation binding) so every cell keeps
the stable, declared control id <column>_<row> (Title_0, Color_2, ...), the
same pattern the working Focus samples 133/189 use with set_focus. The cells
stay two-way editable through absolute model paths into the bound table, and
read_focus parses the cell id the frontend reports in s_focus. Still no
JavaScript shipped with the view.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RjiwGw55EN3XxRMqfKg3eM
---
 src/01/02/z2ui5_cl_demo_app_346.clas.abap | 67 +++++++++++++----------
 1 file changed, 39 insertions(+), 28 deletions(-)

diff --git a/src/01/02/z2ui5_cl_demo_app_346.clas.abap b/src/01/02/z2ui5_cl_demo_app_346.clas.abap
index 3c2c258df..8d0009a67 100644
--- a/src/01/02/z2ui5_cl_demo_app_346.clas.abap
+++ b/src/01/02/z2ui5_cl_demo_app_346.clas.abap
@@ -108,14 +108,14 @@ CLASS z2ui5_cl_demo_app_346 IMPLEMENTATION.
     page->message_strip(
         text     = `Set the keyboard focus to any editable table cell from the backend - type a column id ` &&
                    `(Title, Color, Info, Checkbox or Description) and a row index, then press Set Focus, or ` &&
-                   `use Next / Reset. No JavaScript is shipped with the view: the set_focus follow-up action ` &&
-                   `targets the cell by its control id, and the framework reports the focused cell back in s_focus.`
+                   `use Next / Reset. No JavaScript is shipped with the view: every cell has a stable control ` &&
+                   `id (<column>_<row>) that the set_focus follow-up action targets, and the framework reports ` &&
+                   `the currently focused cell back to the backend in s_focus.`
         type     = `Information`
         showicon = abap_true
         class    = `sapUiSmallMargin` ).
 
     DATA(tab) = page->table(
-            client->_bind_edit( t_tab )
         )->header_toolbar(
             )->overflow_toolbar(
                 )->title( client->_bind( focusid )
@@ -152,29 +152,40 @@ CLASS z2ui5_cl_demo_app_346 IMPLEMENTATION.
         )->column( )->text( `Checkbox` )->get_parent(
         )->column( )->text( `Description` ).
 
-    tab->items(
-        )->column_list_item(
-            )->cells(
-                )->text( `{INDEX}`
-                )->input(
-                    id     = cs_column-title
-                    value  = `{TITLE}`
-                    submit = client->_event( `NEXT` )
-                )->input(
-                    id     = cs_column-color
-                    value  = `{VALUE}`
-                    submit = client->_event( `NEXT` )
-                )->input(
-                    id     = cs_column-info
-                    value  = `{INFO}`
-                    submit = client->_event( `NEXT` )
-                )->checkbox(
-                    id       = cs_column-checkbox
-                    selected = `{CHECKBOX}`
-                )->input(
-                    id     = cs_column-description
-                    value  = `{DESCRIPTION}`
-                    submit = client->_event( `NEXT` ) ).
+    " Build the rows explicitly (no aggregation binding): only then does every
+    " cell keep the stable control id <column>_<row> that set_focus can target.
+    " A bound template would clone the cells under randomly generated ids.
+    DATA(path)  = client->_bind_edit( val = t_tab path = abap_true ).
+    DATA(items) = tab->items( ).
+
+    LOOP AT t_tab REFERENCE INTO DATA(row).
+
+      DATA(i) = sy-tabix - 1.
+
+      items->column_list_item(
+          )->cells(
+              )->text( |{ row->index }|
+              )->input(
+                  id     = |{ cs_column-title }_{ i }|
+                  value  = |\{{ path }/{ i }/TITLE\}|
+                  submit = client->_event( `NEXT` )
+              )->input(
+                  id     = |{ cs_column-color }_{ i }|
+                  value  = |\{{ path }/{ i }/VALUE\}|
+                  submit = client->_event( `NEXT` )
+              )->input(
+                  id     = |{ cs_column-info }_{ i }|
+                  value  = |\{{ path }/{ i }/INFO\}|
+                  submit = client->_event( `NEXT` )
+              )->checkbox(
+                  id       = |{ cs_column-checkbox }_{ i }|
+                  selected = |\{{ path }/{ i }/CHECKBOX\}|
+              )->input(
+                  id     = |{ cs_column-description }_{ i }|
+                  value  = |\{{ path }/{ i }/DESCRIPTION\}|
+                  submit = client->_event( `NEXT` ) ).
+
+    ENDLOOP.
 
     client->view_display( view->stringify( ) ).
 
@@ -183,7 +194,7 @@ CLASS z2ui5_cl_demo_app_346 IMPLEMENTATION.
 
   METHOD focus.
 
-    focusid = |{ focuscolumn }-{ focusrow }|.
+    focusid = |{ focuscolumn }_{ focusrow }|.
 
     client->follow_up_action(
         val   = z2ui5_if_client=>cs_event-set_focus
@@ -194,7 +205,7 @@ CLASS z2ui5_cl_demo_app_346 IMPLEMENTATION.
 
   METHOD read_focus.
 
-    SPLIT client->get( )-s_focus-id AT `-` INTO DATA(col) DATA(row).
+    SPLIT client->get( )-s_focus-id AT `_` INTO DATA(col) DATA(row).
 
     IF row IS NOT INITIAL
         AND row CO `0123456789`