Skip to content

Ensure dialog titlebar window controls are centered#1413

Open
Avengium wants to merge 5 commits intoAzgaar:masterfrom
Avengium:Menu-minimize-and-close-button
Open

Ensure dialog titlebar window controls are centered#1413
Avengium wants to merge 5 commits intoAzgaar:masterfrom
Avengium:Menu-minimize-and-close-button

Conversation

@Avengium
Copy link
Copy Markdown
Contributor

@Avengium Avengium commented May 6, 2026

Refine the alignment of the dialog titlebar window controls to ensure a more polished UI.

Description

Currently, the minimize and close buttons in the dialog titlebar appear slightly off-center, creating a visual inconsistency. This PR updates public/index.css to properly align these controls, ensuring they are centered relative to the titlebar container.

But fix / style CSS for issue #1412 .

Dialog titlebar window controls (Minimize and close) look off-center currently.

With this code, I want the appearance of these buttons to be centered.
@netlify
Copy link
Copy Markdown

netlify Bot commented May 6, 2026

Deploy Preview for afmg ready!

Name Link
🔨 Latest commit b874c87
🔍 Latest deploy log https://app.netlify.com/projects/afmg/deploys/69fcc2e714f39b0008e1636d
😎 Deploy Preview https://deploy-preview-1413--afmg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Azgaar
Copy link
Copy Markdown
Owner

Azgaar commented May 6, 2026

It works, but it would be more correct to fix it in: jquery-ui.css .ui-dialog .ui-dialog-titlebar button {. These styles are set there and require correction.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses UI issue #1412 by adjusting CSS so dialog titlebar window controls (minimize / close) render visually centered within their button bounds, improving consistency across dialogs.

Changes:

  • Updates dialog titlebar button layout to use flexbox centering and tighter typographic defaults.
  • Adds a scoped override intended to neutralize jQuery UI icon positioning within titlebar icon-only buttons.

Comment thread public/index.css
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@Avengium
Copy link
Copy Markdown
Contributor Author

Avengium commented May 6, 2026

So, I have to move this to jquery-ui.css and that's all there is to it?

Avengium added 3 commits May 7, 2026 18:42
Moved the code that deals with: `.ui-dialog .ui-dialog-titlebar button {` to `jquery-ui.css`.
@Azgaar
Copy link
Copy Markdown
Owner

Azgaar commented May 8, 2026

So, I have to move this to jquery-ui.css and that's all there is to it?

Refactor it a bit, it has some weird and probably unwanted styles. I mean the header, not the whole file. At some moment we will replace jQuery UI with a custom solution.

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.

3 participants