From 81f56525dbf59b2fcd177a354e2d00e1e1375b2c Mon Sep 17 00:00:00 2001 From: Rashmi Vemuri Date: Wed, 25 Feb 2026 03:02:54 +0530 Subject: [PATCH 01/11] fix(css): make navbar search input fluid for mobile viewports - Replace fixed width: 16rem with width: 100%; max-width: 16rem - Input now shrinks gracefully on small screens (320px-375px) - Desktop appearance preserved via max-width cap - Fix duplicate background-color declaration - Fix invalid // comment syntax in .search-error rule Part of #691 Signed-off-by: Rashmi Vemuri --- .../frontend/src/pages/Search/components/SearchBar.scss | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/application/frontend/src/pages/Search/components/SearchBar.scss b/application/frontend/src/pages/Search/components/SearchBar.scss index 5c71a549d..17b8001cc 100644 --- a/application/frontend/src/pages/Search/components/SearchBar.scss +++ b/application/frontend/src/pages/Search/components/SearchBar.scss @@ -29,7 +29,8 @@ input { padding: 0.5rem 1rem 0.5rem 2.5rem; - width: 16rem; + width: 100%; + max-width: 16rem; background-color: rgba(var(--card-rgb), 0.5); /* tiny bump */ border: 1px solid rgba(102, 157, 246, 0.267); /* slightly brighter sky blue */ @@ -58,6 +59,6 @@ .search-error { margin-top: 0.25rem; font-size: 0.75rem; - color: #f87171; // red-400 + color: #f87171; /* red-400 */ } } From fa90d5050dc1e059f23884be4398f2a98529b2cc Mon Sep 17 00:00:00 2001 From: Rashmi Vemuri Date: Wed, 25 Feb 2026 03:15:36 +0530 Subject: [PATCH 02/11] fix(css): fix CSS syntax errors in chatbot styles - Fix invalid line-height value (removed stray word "consideration") - Fix invalid // comment syntax in .chat-input and .chat-surface Part of #691 Signed-off-by: Rashmi Vemuri --- application/frontend/src/pages/chatbot/chatbot.scss | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/application/frontend/src/pages/chatbot/chatbot.scss b/application/frontend/src/pages/chatbot/chatbot.scss index f16ca0ea3..6fc5034c8 100644 --- a/application/frontend/src/pages/chatbot/chatbot.scss +++ b/application/frontend/src/pages/chatbot/chatbot.scss @@ -118,7 +118,7 @@ h1.ui.header.chatbot-title { padding: 1rem 1.25rem; box-shadow: 0 6px 18px rgba(0, 0, 0, 0.08); text-align: left; - line-height: 1.6 consideration; + line-height: 1.6; animation: fadeInUp 0.25s ease-out; } @@ -267,7 +267,7 @@ h1.ui.header.chatbot-title { margin-top: 1rem; background-color: #daebdb; - // background: transparent; +/* background: transparent; */ border: 1px solid rgb(190, 214, 232); padding: 0.75rem; border-radius: 12px; @@ -392,7 +392,7 @@ h1.ui.header.chatbot-title { flex-direction: column; height: 100%; - background: #fcfcfa; // same palette as input + background: #fcfcfa; /* same palette as input */ backdrop-filter: blur(6px); border-radius: 18px; From 744cea1a183ef2fc98c8292e1f612869c09cc1d0 Mon Sep 17 00:00:00 2001 From: Rashmi Vemuri Date: Thu, 26 Feb 2026 00:40:38 +0530 Subject: [PATCH 03/11] Fix navbar logo overflow and mobile styles Added styles to fix navbar logo overflow and adjust mobile padding and margins. Signed-off-by: Rashmi Vemuri --- .../assets/scss/pixel/layout/_navbar.scss | 25 +++++++++++++++++++ 1 file changed, 25 insertions(+) diff --git a/application/static/assets/scss/pixel/layout/_navbar.scss b/application/static/assets/scss/pixel/layout/_navbar.scss index 6412eab7c..276b23ed0 100644 --- a/application/static/assets/scss/pixel/layout/_navbar.scss +++ b/application/static/assets/scss/pixel/layout/_navbar.scss @@ -444,3 +444,28 @@ animation: hide-navbar-collapse .2s ease forwards; } } +/* Fix Navbar Logo Overflow & Mobile Padding (Issue #691) */ +@media (max-width: 767px) { + .navbar-brand { + img { + height: 2.5rem; /* Shrink from ~10rem equivalent / 40px base */ + width: auto; + max-width: 8rem; /* Prevent overflow on small screens */ + } + margin-left: 1rem !important; /* Reduce left margin from 4rem */ + margin-right: 1rem; /* Balance spacing */ + } + + .navbar-main { + padding: 0.5rem 1rem; /* Add mobile padding to main navbar to avoid edge-touching */ + } + + .nav-item { + margin-left: 0.5rem; /* Reduce item spacing on mobile for better fit */ + margin-right: 0.5rem; + } + + .navbar-toggler { + margin-right: 0.5rem; /* Better touch target & spacing */ + } +} From 20733f87cce51c51c4091bc3f97125d2fb20f096 Mon Sep 17 00:00:00 2001 From: Rashmi Vemuri Date: Thu, 26 Feb 2026 13:10:12 +0530 Subject: [PATCH 04/11] fix(scss): reduce navbar logo size and margin on mobile (Part of #691) Signed-off-by: Rashmi Vemuri --- .../frontend/src/scaffolding/Header/header.scss | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/application/frontend/src/scaffolding/Header/header.scss b/application/frontend/src/scaffolding/Header/header.scss index 4e3ef7316..3e51fc316 100644 --- a/application/frontend/src/scaffolding/Header/header.scss +++ b/application/frontend/src/scaffolding/Header/header.scss @@ -45,6 +45,15 @@ width: 10rem; display: block; } + + @media (max-width: 767px) { + margin-left: 1rem; + img { + width: auto; + max-width: 7rem; + height: 3rem; + } + } } .navbar__desktop-links { @@ -335,3 +344,4 @@ white-space: nowrap; border: 0; } + From 2099f9745cc48dcfadcb7bb94ac3839793b3c4dc Mon Sep 17 00:00:00 2001 From: Rashmi Vemuri Date: Thu, 26 Feb 2026 13:28:23 +0530 Subject: [PATCH 05/11] Implement mobile styles for CommonRequirementEnumeration Added responsive styles for mobile view. Problem: - .cre-page had no mobile media queries at all - padding: 30px with no mobile override causes content to touch screen edges on small devices - width: 50% on __description creates a ~187px wide text column on 375px screens, making content very cramped - font sizes (2rem heading, 1.2rem sub-heading) are oversized for mobile viewports Fix: - Added @media (max-width: 768px) block at bottom of file - Reduced padding to 1rem on mobile - Expanded __description width to 100% on mobile - Reduced __heading font-size from 2rem to 1.5rem - Reduced __sub-heading font-size from 1.2rem to 1rem Desktop layout is completely untouched. Only fires on screens 768px and below. Signed-off-by: Rashmi Vemuri --- .../commonRequirementEnumeration.scss | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/application/frontend/src/pages/CommonRequirementEnumeration/commonRequirementEnumeration.scss b/application/frontend/src/pages/CommonRequirementEnumeration/commonRequirementEnumeration.scss index 2c92d5c5c..f650ccd28 100644 --- a/application/frontend/src/pages/CommonRequirementEnumeration/commonRequirementEnumeration.scss +++ b/application/frontend/src/pages/CommonRequirementEnumeration/commonRequirementEnumeration.scss @@ -72,3 +72,21 @@ } } + +@media (max-width: 768px) { + .cre-page { + padding: 1rem; /* Reduce from 30px to prevent content touching edges */ + + &__description { + width: 100%; /* Expand from 50% - too narrow on small screens */ + } + + &__heading { + font-size: 1.5rem; /* Reduce from 2rem - oversized on mobile */ + } + + &__sub-heading { + font-size: 1rem; /* Reduce from 1.2rem - proportional for mobile */ + } + } +} From 3ea60922c873bca0c3fbe179ff473b2223d407f7 Mon Sep 17 00:00:00 2001 From: Rashmi Vemuri Date: Fri, 27 Feb 2026 00:15:29 +0530 Subject: [PATCH 06/11] fix(scss): add mobile padding and responsive fixes to Explorer page (Part of #691) Adjusted styles for mobile responsiveness, including padding and width changes for search field and item margins. Problem: - main#explorer-content had padding: 30px with no mobile override - search input fixed at 320px overflows on screens < 375px - .item had margin-left: 40px causing content overflow on mobile Fix: - Added @media (max-width: 768px) block at bottom of file - Reduced padding to 1rem on mobile - Expanded search input width to 100% on mobile - Reduced .item left margin to 1rem on mobile Desktop layout completely untouched. Signed-off-by: Rashmi Vemuri --- .../frontend/src/pages/Explorer/explorer.scss | 15 +++++++++++++++ 1 file changed, 15 insertions(+) diff --git a/application/frontend/src/pages/Explorer/explorer.scss b/application/frontend/src/pages/Explorer/explorer.scss index dae6bf27a..384671509 100644 --- a/application/frontend/src/pages/Explorer/explorer.scss +++ b/application/frontend/src/pages/Explorer/explorer.scss @@ -110,3 +110,18 @@ main#explorer-content { flex-direction: column; } } +@media (max-width: 768px) { + main#explorer-content { + padding: 1rem; /* Reduce from 30px to prevent content touching edges */ + + .search-field { + input { + width: 100%; /* Expand from fixed 320px - overflows on small screens */ + } + } + + .item { + margin: 4px 4px 4px 1rem; /* Reduce from 40px left margin on mobile */ + } + } +} From 67264211be777c1b863863c4b2e9ea22214403a7 Mon Sep 17 00:00:00 2001 From: Rashmi Vemuri Date: Fri, 27 Feb 2026 00:21:52 +0530 Subject: [PATCH 07/11] Add responsive styles for standard page on mobile Problem: - .standard-page had padding: 30px with no mobile override - font sizes (2rem heading, 1.2rem sub-heading) are oversized for mobile viewports - existing media query only handled word-break, not padding or font sizes Fix: - Added @media (max-width: 768px) block at bottom of file - Reduced padding to 1rem on mobile - Reduced __heading font-size from 2rem to 1.5rem - Reduced __sub-heading font-size from 1.2rem to 1rem Desktop layout completely untouched. Signed-off-by: Rashmi Vemuri --- .../frontend/src/pages/Standard/standard.scss | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/application/frontend/src/pages/Standard/standard.scss b/application/frontend/src/pages/Standard/standard.scss index caa635923..e7c6496e6 100644 --- a/application/frontend/src/pages/Standard/standard.scss +++ b/application/frontend/src/pages/Standard/standard.scss @@ -32,3 +32,16 @@ word-break: break-word; } } +@media (max-width: 768px) { + .standard-page { + padding: 1rem; /* Reduce from 30px to prevent content touching edges */ + + &__heading { + font-size: 1.5rem; /* Reduce from 2rem - oversized on mobile */ + } + + &__sub-heading { + font-size: 1rem; /* Reduce from 1.2rem - proportional for mobile */ + } + } +} From 6c2a68ab7031a80bdc7fda6ffd796a41cd00e5b5 Mon Sep 17 00:00:00 2001 From: Rashmi Vemuri Date: Fri, 27 Feb 2026 00:25:30 +0530 Subject: [PATCH 08/11] Add media query for responsive padding adjustment Adjusted padding for gap analysis on smaller screens. Signed-off-by: Rashmi Vemuri --- application/frontend/src/pages/GapAnalysis/GapAnalysis.scss | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/application/frontend/src/pages/GapAnalysis/GapAnalysis.scss b/application/frontend/src/pages/GapAnalysis/GapAnalysis.scss index d15bf0089..feccf0672 100644 --- a/application/frontend/src/pages/GapAnalysis/GapAnalysis.scss +++ b/application/frontend/src/pages/GapAnalysis/GapAnalysis.scss @@ -15,3 +15,8 @@ main#gap-analysis { } } } +@media (max-width: 768px) { + main#gap-analysis { + padding: 1rem; /* Reduce from 30px to prevent content touching edges */ + } +} From ee75bf3c03a58fc074553f8e5672fea0727c2ebb Mon Sep 17 00:00:00 2001 From: Rashmi Vemuri Date: Tue, 3 Mar 2026 01:01:07 +0530 Subject: [PATCH 09/11] Add instructions for starting services locally Added instructions for starting required services before running the application. Signed-off-by: Rashmi Vemuri --- docs/developmentSetup.md | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/docs/developmentSetup.md b/docs/developmentSetup.md index ae0fda6a5..501f8f322 100644 --- a/docs/developmentSetup.md +++ b/docs/developmentSetup.md @@ -113,6 +113,10 @@ You can run `make migrate-upgrade` first and then `make upstream-sync` ## Running locally +First, start the required services (Redis and Neo4j) with: + +`make start-containers` + You can run the backend with `make dev-flask`. At the time of writing the backend URL is `http://localhost:5000` by default. You can run the frontend with `yarn start`. This should open a browser tab at the application's front page and also automatically reload the page whenever changes are detected. At the time of writing the frontend URL is `http://localhost:9001` by default. From 8842fee8ccc0d516a32fe8e2de752c9bb8d619e1 Mon Sep 17 00:00:00 2001 From: Rashmi Vemuri Date: Tue, 3 Mar 2026 01:13:28 +0530 Subject: [PATCH 10/11] Check Redis availability before gap analysis Added error handling for Redis connection failure. Signed-off-by: Rashmi Vemuri --- application/utils/gap_analysis.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/application/utils/gap_analysis.py b/application/utils/gap_analysis.py index da2bc246e..4d5e4df8d 100644 --- a/application/utils/gap_analysis.py +++ b/application/utils/gap_analysis.py @@ -129,6 +129,9 @@ def schedule(standards: List[str], database): logger.info(f"Gap analysis result for {standards_hash} does not exist") conn = redis.connect() + if conn is None: + logger.error("Redis is not available. Please run 'make start-containers' first.") + return {"error": "Redis is not available. Please run 'make start-containers' first."} gap_analysis_results = conn.get(standards_hash) if ( gap_analysis_results From 9d96bca6f19ab841b106cbdd5be13e6cd7b47164 Mon Sep 17 00:00:00 2001 From: Rashmi Vemuri Date: Tue, 3 Mar 2026 01:22:00 +0530 Subject: [PATCH 11/11] Refactor error logging for Redis connection failure Signed-off-by: Rashmi Vemuri --- application/utils/gap_analysis.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/application/utils/gap_analysis.py b/application/utils/gap_analysis.py index 4d5e4df8d..9e3dab04d 100644 --- a/application/utils/gap_analysis.py +++ b/application/utils/gap_analysis.py @@ -130,8 +130,12 @@ def schedule(standards: List[str], database): conn = redis.connect() if conn is None: - logger.error("Redis is not available. Please run 'make start-containers' first.") - return {"error": "Redis is not available. Please run 'make start-containers' first."} + logger.error( + "Redis is not available. Please run 'make start-containers' first." + ) + return { + "error": "Redis is not available. Please run 'make start-containers' first." + } gap_analysis_results = conn.get(standards_hash) if ( gap_analysis_results