19 lines
1.3 KiB
Plaintext
19 lines
1.3 KiB
Plaintext
Plan:
|
|
|
|
1. Identified a critical HTML structure error: a missing `</tbody>` tag in the `accounting` journal table (index.php) was causing the browser to render the page incorrectly, potentially hiding the main content area.
|
|
2. Found a JavaScript logic error in the `renderPagination` function where `document.currentScript` was being accessed inside a `DOMContentLoaded` listener (where it is null), causing the "Rows per page" dropdown to remain at the bottom instead of moving above the table.
|
|
3. Fixed both issues by patching `index.php`.
|
|
|
|
Changed:
|
|
|
|
* `index.php`:
|
|
* Added the missing `</tbody>` tag to the accounting journal table.
|
|
* Updated the `renderPagination` inline script to correctly capture `document.currentScript` immediately and safely move the "Rows per page" dropdown above the table wrapper.
|
|
|
|
Notes:
|
|
|
|
* The missing `</tbody>` tag likely caused the browser to treat the entire main content area as invalid or hidden, explaining why only the sidebar was visible.
|
|
* The pagination dropdown should now appear correctly above the table.
|
|
|
|
Next: Please verify that the accounting page (`/index.php?page=accounting`) now loads correctly with the sidebar AND the main content table. Also check if the "Rows per page" dropdown is visible above the table.
|