• 
      

    Include the attachment point in the action's DOM element ID.

    Review Request #14674 — Created Nov. 6, 2025 and submitted

    Information

    Review Board
    release-7.1.x

    Reviewers

    DOM element IDs now include the attachment point ID as part, ensuring
    that two views for the same action in different attachment points won't
    conflict.

    In the process, BaseAction.get_dom_element_id() has been deprecated
    and returns an empty string. This will still be used if overridden,
    but it's deprecated.

    Unit tests pass.

    Tested all the actions in the UI.

    Summary ID
    Include the attachment point in the action's DOM element ID.
    DOM element IDs now include the attachment point ID as part, ensuring that two views for the same action in different attachment points won't conflict. In the process, `BaseAction.get_dom_element_id()` has been deprecated and returns an empty string. This will still be used if overridden, but it's deprecated.
    4cc70a8a6766408d688bacca989155443879c67d
    david
    1. Ship It!
    2. 
        
    maubin
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-7.1.x (34465e2)