search

Bruno17 / MIGX

MIGX for revo 2.2 and above
83 stars 78 forks source link

Mulitiple uses of the same element ID #362

Closed jmacdonald-spectacle closed 3 years ago

jmacdonald-spectacle commented 3 years ago

This isn't a problem for me, just an observation.

When I was looking at the generated code for an opened MIGX window, I found the following run:

<div id="modx-window-mi-grid-update-26-tabs" class=" x-tab-panel modx-tabs x-tab-panel-noborder"> 
    <div id="modx-window-mi-grid-update-26-tabs">
        <div id="modx-window-mi-grid-update-26-tabs">
            <div id="modx-window-mi-grid-update-26-tabs">
                <div id="modx-window-mi-grid-update-26-tabs"> 
                    <div id="modx-window-mi-grid-update-26-tabs">

Just thought I'd mention it in case there's some runaway code.

MIGX: 2.13.0-rc1 MODX: 2.8.1