fix(cli:templates): improve side nav auth template UI and selection flow#1725
Open
georgianastasov wants to merge 47 commits into
Open
fix(cli:templates): improve side nav auth template UI and selection flow#1725georgianastasov wants to merge 47 commits into
georgianastasov wants to merge 47 commits into
Conversation
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…ion) for React and WC
…com/IgniteUI/igniteui-cli into ganastasov/align-side-nav-templates
…ini-side-nav-template
…e improved defalt side-nav
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…ini-side-nav-template
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refines the Angular Side Navigation authentication template UI (pre/post login) and updates the CLI wizard so authentication is offered as an add-on option after choosing a base navigation template (instead of listing a standalone “Side Nav + Auth” template).
Changes:
- Hide
*-authtemplates from the main template list and add a follow-up “add authentication?” confirmation step in the wizard. - Improve Side Nav Auth generated UI: navbar login/avatar behavior, drawer items/icons, and a more polished profile page + updated login/register form styling.
- Add a hidden
side-nav-mini-authvariant composed fromside-nav-mini+ shared auth overlay with mini-specific app shell overrides.
Reviewed changes
Copilot reviewed 28 out of 28 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/igx-templates/igx-ts/projects/side-nav/files/src/app/app.ts | Introduces appTitle property to bind navbar title via input binding. |
| packages/igx-templates/igx-ts/projects/side-nav/files/src/app/app.html | Switches navbar title to property binding ([title]) to avoid native tooltip behavior. |
| packages/igx-templates/igx-ts/projects/side-nav-mini-auth/index.ts | Adds hidden mini side-nav + auth project template composed via overlay paths. |
| packages/igx-templates/igx-ts/projects/side-nav-mini-auth/files/src/app/app.ts | New combined mini side-nav shell that switches drawer links based on auth state. |
| packages/igx-templates/igx-ts/projects/side-nav-mini-auth/files/src/app/app.spec.ts | Adds basic app creation test for the new mini-auth template. |
| packages/igx-templates/igx-ts/projects/side-nav-mini-auth/files/src/app/app.html | New mini-auth app shell template with navbar + login bar + mini drawer templates. |
| packages/igx-templates/igx-ts/projects/side-nav-mini-auth/files/src/app/app.config.ts | New app config that wires router + Ignite UI modules + auth providers. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/index.ts | Hides the standalone side-nav-auth template from the main selection list. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/services/fake-backend.ts | Adds prominent dev-only guidance comment for the fake backend interceptor. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/services/external-auth-configs.ts | Adds clarifying comment about auth base path and redirect URI construction. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/register/register.ts | Updates input group prefix->suffix usage to match updated markup. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/register/register.scss | Restyles register form layout, input group tokens, and button/link presentation. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/register/register.html | Updates register form markup (border inputs, suffix icons, button-based navigation). |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/profile/profile.scss | Adds a new polished profile page layout and responsive styling. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/profile/profile.html | Replaces placeholder “logged in” text with a structured profile view. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/login/login.ts | Updates input group prefix->suffix usage and minor cleanup. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/login/login.scss | Restyles login form layout, button/link presentation, and social login section. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/login/login.html | Updates login form markup and button styling (incl. external login buttons). |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/login-dialog/login-dialog.scss | Improves login dialog sizing/padding for small screens. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/login-bar/login-bar.ts | Updates navbar auth UI with overlay positioning settings and profile-route selection. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/login-bar/login-bar.spec.ts | Aligns tests with updated login button/avatar DOM structure. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/login-bar/login-bar.scss | Adds theming tokens and updated styles for login button, avatar, and dropdown. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/authentication/login-bar/login-bar.html | Reworks logged-in UI from “avatar inside button” to clickable avatar + dropdown styling. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/app.ts | Adds appTitle, drawer icons, and switches nav links based on auth state. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/app.routes.ts | Adds icon metadata to the home route for drawer rendering. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/app.html | Aligns auth side-nav layout with regular side-nav template structure and icon rendering. |
| packages/igx-templates/igx-ts/projects/side-nav-auth/files/src/app/app.config.ts | Adds inline guidance/comments for configuring social login providers. |
| packages/core/prompt/BasePromptSession.ts | Adds an “add authentication?” follow-up prompt when a *-auth variant exists. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
… auth variant selection
11 tasks
…soft for React and WC
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR improves the Angular Side Navigation with Authentication template UI and aligns it more closely with the regular Side Navigation template behavior.
The changes polish the generated authentication experience before and after login, including the navbar login button contrast, project title tooltip behavior, responsive resource link layout, logged-in avatar/dropdown behavior, side navigation active item, and the default profile page design.
This PR also updates the template selection flow so authentication is treated as an additional option after selecting a navigation template, instead of being exposed as a separate standalone Side Navigation with Authentication template. This makes the generated auth template easier to discover and helps avoid maintaining duplicate navigation template structures.
Related Issue
Closes #1724
Type of Change
Affected Packages
igniteui-cli(packages/cli)@igniteui/cli-core(packages/core)@igniteui/angular-templates(packages/igx-templates)@igniteui/angular-schematics(packages/ng-schematics)@igniteui/mcp-server(packages/igniteui-mcp)Checklist
npm run test)npm run build)npm run lint)Additional Context
The issue was visible in the generated Angular Side Navigation with Authentication application UI.



