Skip to content

Add Slugify Example#313

Open
joshuauaua wants to merge 16 commits intoIvy-Interactive:mainfrom
joshuauaua:feature/slugify
Open

Add Slugify Example#313
joshuauaua wants to merge 16 commits intoIvy-Interactive:mainfrom
joshuauaua:feature/slugify

Conversation

@joshuauaua
Copy link

This is a demo package project for the Slugify nuget package (https://www.nuget.org/packages/Slugify.Core/5.1.1/). The project uses the latest version of Slugify (5.1.1) and demonstrates how to integrate it with the Ivy framework to convert text into URL-friendly “slugs”.

Features:
• Convert Text to Slugs: Transform any text input into a URL-friendly slug
• Customizable Options:
1. Force lowercase – ensure all characters are lowercase.
2. Collapse whitespace – combine multiple spaces into one.
3. Collapse dashes – remove consecutive dashes.
4. Trim whitespace – remove leading and trailing spaces.

Screenshot of demo :
Screenshot 2025-11-18 at 15 07 08

joshuauaua and others added 8 commits November 18, 2025 14:34
Copilot AI review requested due to automatic review settings November 18, 2025 07:34
Copy link
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 adds a new demo project showcasing the Slugify.Core NuGet package (v5.1.1) integrated with the Ivy framework. The demo provides an interactive web application for converting text into URL-friendly slugs with customizable options.

Key Changes:

  • New Slugify demo application with interactive UI for slug generation
  • Four configurable options: force lowercase, collapse whitespace, collapse dashes, and trim whitespace
  • Complete project setup including .NET 9.0 configuration, Docker support, and deployment instructions

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
Slugify.csproj Project configuration with .NET 9.0, Ivy framework v1.0.135.0, and Slugify.Core v5.1.1 dependencies
Program.cs Application entry point with Chrome UI configuration and server initialization
GlobalUsings.cs Global using statements for Ivy framework and Slugify namespaces
Apps/SlugApp.cs Main slug generator UI with input fields, option toggles, and result display
README.md Documentation covering setup, features, deployment instructions, and usage guide
Dockerfile Multi-stage Docker build configuration for .NET 9.0 deployment
.gitignore Standard .NET project ignore patterns for build artifacts and IDE files
.dockerignore Docker build exclusions for development files and build artifacts

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

joshuauaua and others added 7 commits November 18, 2025 21:50
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…y default by the library so does not need to be defined (leads to an error)
@rorychatt
Copy link
Collaborator

@ArtemLazarchuk Please review this user-submitted project

@ArtemLazarchuk
Copy link
Collaborator

@joshuauaua The project looks great, and the functionality works correctly overall. Only a few minor improvements are needed, which I described above.
One more suggestion: remove the Expandable, as it’s not really necessary here — only one example is displayed, which we expect, so it would be better to see it immediately without any extra actions. Also, in my opinion, the input and output can be placed in a single card, which would make the interface more convenient and clear.

-Placed all elements into one single card instead of two
@joshuauaua
Copy link
Author

@ArtemLazarchuk I have now committed the following changes from your comment:
-Removed Expandable to display result immediately
-Placed all elements into a single card for cleaner design

@ArtemLazarchuk
Copy link
Collaborator

@ArtemLazarchuk I have now committed the following changes from your comment: -Removed Expandable to display result immediately -Placed all elements into a single card for cleaner design

see my feedback above

@rorychatt
Copy link
Collaborator

image

the ultimate c-sharp framework фывшнг фцрвгло

Copy link
Collaborator

@rorychatt rorychatt left a comment

Choose a reason for hiding this comment

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

As we discussed during our call, see if you can make this look more like other projects.

We have a checklist here
#218

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

4 participants