Minor UX accessibility polish + local dev reliability updates#133
Open
eccentriccoder01 wants to merge 1 commit intosoft-eng-practicum:developfrom
Open
Minor UX accessibility polish + local dev reliability updates#133eccentriccoder01 wants to merge 1 commit intosoft-eng-practicum:developfrom
eccentriccoder01 wants to merge 1 commit intosoft-eng-practicum:developfrom
Conversation
…errides for type dependencies style: enhance home component styles for nav-cta focus visibility style: add focus-visible styles for nav-cta in foundation styles chore: add proxy configuration for API requests
eccentriccoder01
force-pushed
the
develop
branch
from
f33d9a1 to
04c0696
Compare
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
1 participant
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.
This PR bundles a small set of minor improvements I made while setting up and validating the local development flow, plus a focused homepage CTA accessibility polish.
Summary
This is intentionally a minor-scope PR with two goals:
What changed
Improved CTA contrast on homepage blue sections and ensured CTA text stays readable.
File: src/Analysim.Web/ClientApp/src/app/home/home.component.scss
Added keyboard-visible focus style for CTA buttons (
:focus-visible) to improve accessibility and navigation feedback.File: src/Analysim.Web/ClientApp/src/assets/styles/utilities/_foundation.scss
Added Angular dev proxy config for
/apito backend (https://localhost:5001) to avoid local API routing/CORS friction during frontend dev.File: src/Analysim.Web/ClientApp/proxy.conf.json
Updated frontend start script to use proxy config; pinned type packages for compatibility in local environment.
File: src/Analysim.Web/ClientApp/package.json
Lockfile update corresponding to dependency changes above.
File: src/Analysim.Web/ClientApp/package-lock.json
Exposed and initialized local PostgreSQL defaults in Compose for smoother DB startup/migration flow.
File: docker-compose.yml
Why this PR is still minor
Scope note
The UX-facing part of this PR is limited to homepage CTA visibility/focus behavior.
The rest are dev-environment reliability changes that reduce setup friction.
Screenshots
Testing done
@cengique @adolocm Kindly review the changes and merge it if appropriate