fix/feat(frontend): Fix container appearing outside of viewport & improvement on responsive layouts #223
Conversation
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
Improved the mobile and tablet user experience with a more compact and usable layout. Fixed - the output text container was appearing outside of viewport for certain screen sizes UI/UX Changes for tablet and mobile viewports. Original and default layout remains the same: - CSS fix for overflow and displaying containers outside of viewport - Set up responsive design with media queries - Reordered buttons for better cohesion with cogstack color theme: Deidentify and Use Example buttons first, then Redact checkbox below - Reduced padding and gaps throughout mobile layout for more compact design - Fixed scrolling issues on viewports <1280px by properly constraining viewport-container height - Reduced border sizes on mobile from 15px to 5px for cleaner appearance - Reorganised mobile middle container for logo to be a watermark background Bug fix - Removed unused import that called defunct variable (deid_text) from views.py
mart-r
added a commit
that referenced
this pull request
Nov 18, 2025
The deid_text method in medcat.utils.ner was deprecated in favour of using the instead. However, the change in #223 removed the import (probably because otherwise it failed due to an import error) without replacing it with the new alteranative. This PR fixes that.
mart-r
added a commit
that referenced
this pull request
Nov 18, 2025
The deid_text method in medcat.utils.ner was deprecated in favour of using the instead. However, the change in #223 removed the import (probably because otherwise it failed due to an import error) without replacing it with the new alteranative. This PR fixes that.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Summary
This is a bugfix that lead into minor improvements on responsive layouts for the AnonCAT demo app.
Problem: the output text container was appearing outside of viewport for certain screen sizes
Fix: Retaining mostly the default layout, with UI/UX Changes for tablet and mobile viewports.
Changes
Additional bug fix
Screenshots
Before (Desktop)
After (Desktop)
After (Mobile)
Test plan
Additional Notes
All changes are CSS-only for the responsive layouts. No functional changes to the deidentification logic.
Discussion whether to update this to medcat v2 as well? currently on 1.1
At some point to update the currently deployed version that is on internal infra.