Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 14, 2026

On smaller viewports, ion-split-pane shrinks the menu to 28% width (Ionic default) while content padding remains fixed at 360px, creating a gap.

Changes

  • Override --side-max-width with --ios26-menu-width on ion-split-pane to maintain consistent width
  • Apply same override in demo for consistency
ion-split-pane.ios:not(.ios26-disabled) {
  --side-max-width: var(--ios26-menu-width);
}

Before

Gap between menu and content

After

1024px viewport:
Fixed alignment

992px viewport:
Fixed alignment

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • www.google-analytics.com
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/ionic-theme-ios26/ionic-theme-ios26/demo/node_modules/.bin/ng build --configuration=production (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>Side menu dynamic size causes increased gap</issue_title>
<issue_description>In smaller viewports, side menu goes thinner and this causes an increased gap between the page content and side menu.

Image

I think the easiest way to fix this is overriding Ionic's default 28% max-width definition (via SCSS variable) explicitly, like:

ion-split-pane {
  --side-max-width: var(--ios26-menu-width);
}

</issue_description>

<agent_instructions>demoもサイドメニューの横幅を変えるようにして</agent_instructions>

Comments on the Issue (you are @copilot in this section)

@rdlabo Thanks! You’re absolutely right. I’ll integrate this into the implementation soon.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@netlify
Copy link

netlify bot commented Jan 14, 2026

Deploy Preview for ionic-theme-ios26 ready!

Name Link
🔨 Latest commit e42c0bf
🔍 Latest deploy log https://app.netlify.com/projects/ionic-theme-ios26/deploys/696738330575f90008f62839
😎 Deploy Preview https://deploy-preview-90--ionic-theme-ios26.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.

Copilot AI and others added 3 commits January 14, 2026 06:00
Co-authored-by: rdlabo <9690024+rdlabo@users.noreply.github.com>
Co-authored-by: rdlabo <9690024+rdlabo@users.noreply.github.com>
Co-authored-by: rdlabo <9690024+rdlabo@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix gap between content and side menu in smaller viewports Fix split-pane side menu gap on smaller viewports Jan 14, 2026
Copilot AI requested a review from rdlabo January 14, 2026 06:07
@rdlabo rdlabo marked this pull request as ready for review January 14, 2026 06:18
@github-actions
Copy link
Contributor

Playwright test results

failed  76 failed
passed  2 passed

Details

stats  78 tests across 1 suite
duration  8 minutes, 22 seconds
commit  7962ba0
info  Please check the workflow run for details. And if you want to update the screenshots, please comment with /update-screenshots.

Failed tests

chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for button
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for toast:top
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for popover
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for toggle
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for alert:all
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for checkbox
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for modal:normal
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for select:alert
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for card
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for modal:sheet
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for accordion
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for date-and-time-pickers
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for select:popover
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for select:modal
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for progress-indicators:message
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for toast:bottom
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for toolbar
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for alert:remove-app
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for tabs
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for breadcrumbs
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for toast:anchor
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for radio
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for select:action-sheet
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for chip
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for toast:middle
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for searchbar
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for action-sheet:all
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for action-sheet:button-only
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for alert:button-only
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for inputs
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for range
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for progress-indicators:icon-only
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for floating-action-button
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for segment
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for item-list
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for reorder
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for action-sheet:no-cancel
chromium › screenshot.spec.ts › Screenshot Tests - All Routes › should match screenshot for alert:no-cancel
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for button
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for toast:top
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for popover
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for toggle
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for alert:all
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for checkbox
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for modal:normal
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for select:alert
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for card
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for modal:sheet
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for accordion
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for date-and-time-pickers
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for select:popover
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for select:modal
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for progress-indicators:message
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for toast:bottom
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for toolbar
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for alert:remove-app
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for tabs
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for breadcrumbs
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for toast:anchor
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for radio
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for select:action-sheet
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for chip
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for toast:middle
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for searchbar
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for action-sheet:all
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for action-sheet:button-only
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for alert:button-only
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for inputs
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for range
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for progress-indicators:icon-only
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for floating-action-button
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for segment
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for item-list
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for reorder
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for action-sheet:no-cancel
chromium › screenshot.spec.ts › Screenshot Tests - Dark Mode › should match dark mode screenshot for alert:no-cancel

@rdlabo
Copy link
Contributor

rdlabo commented Jan 14, 2026

/update-screenshots

@github-actions
Copy link
Contributor

📊 Playwright Test Report

View the detailed test report: https://rdlabo-team.github.io/ionic-theme-ios26/pr-90/

github-actions bot added a commit that referenced this pull request Jan 14, 2026
@github-actions
Copy link
Contributor

✅ Screenshots have been updated successfully!

The new screenshots have been committed to this PR.

@rdlabo rdlabo merged commit 3b2f208 into main Jan 14, 2026
4 checks passed
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.

Side menu dynamic size causes increased gap

2 participants