Skip to content

[pull] main from nodejs:main - #1868

Merged
pull[bot] merged 1 commit into
NOUIY:mainfrom
nodejs:main
Mar 15, 2026
Merged

pull[bot] merged 1 commit into
NOUIY:mainfrom
nodejs:main

Conversation

@pull

@pull pull Bot commented Mar 15, 2026 •

Copy link
Copy Markdown

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

* chore: update next-env.d.ts

* feat: minimize mobile nav empty space (#8719)

Only render the navItems container when the array has items to prevent
an empty padded container from creating unwanted space in the mobile navigation.

* Update import path for routes type definitions

Signed-off-by: Igor Rodrigues Machado <107290125+igorrm19@users.noreply.github.com>

* Update packages/ui-components/src/Containers/NavBar/index.tsx

Co-authored-by: Claudio Wunder <cwunder@gnome.org>
Signed-off-by: Igor Rodrigues Machado <107290125+igorrm19@users.noreply.github.com>

* Fix conditional rendering of navItems in NavBar

Is this breaking the build with navItems?.length > 0 &&

Signed-off-by: Igor Rodrigues Machado <107290125+igorrm19@users.noreply.github.com>

* Remove blank line in NavBar component

Removed unnecessary blank line before the main navigation items div.

Signed-off-by: Igor Rodrigues Machado <107290125+igorrm19@users.noreply.github.com>

* style: fix prettier formatting in NavBar

---------

Signed-off-by: Igor Rodrigues Machado <107290125+igorrm19@users.noreply.github.com>
Co-authored-by: Claudio Wunder <cwunder@gnome.org>
@pull pull Bot locked and limited conversation to collaborators Mar 15, 2026
@pull pull Bot added the ⤵️ pull label Mar 15, 2026
@pull
pull Bot merged commit 49b4f39 into NOUIY:main Mar 15, 2026

This branch had an error being deployed

1 failed deployment
Storybook — 49b4f391 Deployed Mar 15, 2026 by pull[bot] via Chromatic #212
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant