Skip to content

Conversation

@skerbis
Copy link
Member

@skerbis skerbis commented Mar 16, 2025

Update for Adminer 5.0.5 Namespace Compatibility and UI Improvements

Overview

This PR updates the Adminer addon to work with Adminer 5.0.5, which introduced a new namespace structure. Additionally, we've enhanced the rex_sql_table code display with a modern, responsive design including dark mode support.

Changes

Namespace Compatibility

  • Updated rex_adminer class to extend \Adminer\Adminer (with namespace) instead of just Adminer
  • Fixed script integration to work with Adminer's new namespace structure
  • Updated nonce handling to use Adminer's namespace-specific functions

UI Improvements

  • Enhanced the rex_sql_table code display with:
    • CSS variables for better maintainability and theming
    • Dark mode support with toggle button
    • User preference saving via localStorage
    • Improved typography with better monospace font stack
    • Modern styling with rounded corners, subtle shadows, and smooth transitions
    • Better positioning and padding for improved readability

Security Enhancements

  • Properly integrated with Adminer's Content Security Policy mechanisms
  • Ensured all inline scripts have proper nonce attributes

Testing

The changes have been tested with:

  • REDAXO 5.18.2
  • PHP 8.2.27
  • MariaDB 10.11.11

All functionality works as expected, including database management, multiple database support, and the rex_sql_table code generation feature.

Screenshots

(Optional: Add screenshots showing the new dark mode and improved design)

Potential Impact

This update is required for compatibility with newer versions of Adminer. Users upgrading should experience no functional changes, but will notice the improved code display interface.

@skerbis skerbis merged commit 58b981e into master Mar 16, 2025
@skerbis skerbis deleted the 5 branch March 16, 2025 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants