Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update searchQuery on input change to handle Enter key press correctly #105

Merged
merged 23 commits into from
Jun 17, 2024

Conversation

miladsoft
Copy link
Member

No description provided.

- Replaced @Bind with @oninput for searchQuery in the input field to ensure immediate update.
- Added UpdateSearchQuery method to handle input changes.
- Ensured searchQuery has the correct value when Enter key is pressed.
- Configure GitHub Actions to deploy Angor project to two different domains using separate branches.
- Minify JS and CSS files before deployment.
- Include `.nojekyll` and `CNAME` files for each domain.
- Publish to `gh-pages-test` for test.angor.io and `gh-pages-beta` for beta.angor.io.
@itailiors
Copy link
Collaborator

Cypress regression tests are failing because they search for the theme icon to enable dark mode at the start of the run. @miladsoft , once you are done, I'll update the test to use a data-cy attribute and ensure there are no actions searching for a specific icon (as there shouldn't be any).

@miladsoft
Copy link
Member Author

Cypress regression tests are failing because they search for the theme icon to enable dark mode at the start of the run. @miladsoft , once you are done, I'll update the test to use a data-cy attribute and ensure there are no actions searching for a specific icon (as there shouldn't be any).

fixed

Implemented a new IconService class to dynamically handle SVG icons in the Angor App project. This service fetches SVG icons from the wwwroot/assets/icons folder, adjusts their size, and updates their stroke and fill colors as needed.

Features:
- Fetches SVG icons from the server dynamically based on icon name.
- Adjusts the width and height of the SVG icons to specified dimensions.
- Updates the stroke and fill colors of the SVG icons to the provided values.
- Caches the icons for improved performance and reduced server requests.

Benefits:
- Centralized management of SVG icons, ensuring consistency across the application.
- Enhanced performance through caching, reducing redundant server requests.
- Improved flexibility in icon customization, allowing dynamic adjustments to icon size and colors.
- Simplified code in components by offloading SVG handling to a dedicated service.
- Added all SVG icons in wwwroot/assets/icons as embedded resources.
- Updated IconService to load icons from embedded resources.
- Modified .csproj file to include SVG icons as embedded resources.
- Improved performance and offline capabilities by eliminating server dependency for icons.
remove extra code:
builder.Services.AddScoped(sp => new HttpClient { BaseAddress = new Uri(builder.HostEnvironment.BaseAddress) });
Copy link
Member

@dangershony dangershony left a comment

Choose a reason for hiding this comment

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

How bigger is the app if we pull the icons from resource?

@miladsoft
Copy link
Member Author

How bigger is the app if we pull the icons from resource?

about 5 kb

@dangershony
Copy link
Member

How bigger is the app if we pull the icons from resource?

about 5 kb

Ok I suppose this is not a big deal

@dangershony
Copy link
Member

@itailiors when you have a free moment check why the test fails and fix it

@dangershony dangershony merged commit 2730a0a into main Jun 17, 2024
3 of 4 checks passed
@dangershony dangershony deleted the UI/UX branch June 17, 2024 15:17
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