Jan Fixes and Enhancements#470
Merged
kshitijrajsharma merged 41 commits intohotosm:developfrom Feb 4, 2026
Merged
Conversation
- Created ContentIcon component for content representation. - Added DurationIcon component to represent duration visually. - Implemented GlobeIcon component for global representation.
…tion - Implemented a new Carousel component with responsive design using Shoelace. - Added styles for carousel and update cards in carousel.css. - Created tests for the getSlidesPerPage function to ensure correct slide counts based on breakpoints. - Developed useBreakpoint hook tests to verify breakpoint detection and updates. - Introduced useFairUpdates hook to fetch updates from a specified URL with error handling.
Learn Page UI Update
Member
There are still lot of history API calls in the start mapping page , it is bombbarding browser within short period of time , is there a way to control it ?
|
kshitijrajsharma
requested changes
Feb 4, 2026
Member
kshitijrajsharma
left a comment
There was a problem hiding this comment.
I couldn't verify the bug fixes, one more iteration would be needed
| export const extractYouTubeVideoId = (url: string): string | null => { | ||
| // Regex to match various YouTube URL formats | ||
| const youtubeRegex = | ||
| /(?:https?:\/\/)?(?:www\.)?(?:youtube\.com\/(?:watch\?v=|embed\/|v\/)|youtu\.be\/)([\w-]{11})/i; |
Member
There was a problem hiding this comment.
This seems problematic !
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
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.
What does this PR do ?
This PR includs some features and bugs raised on Slack.
Screenshots
Screen.Recording.2026-01-18.at.21.06.44.mov
Screen.Recording.2026-01-18.at.21.05.21.mov
Screen.Recording.2026-01-18.at.21.03.58.mov
Before - Lots's of CORS error because OSM does not provide tiles at zoom 20+.
Screen.Recording.2026-01-18.at.21.08.45.mov
After - Making maxzoom to be 19 prevents this from happening.
Screen.Recording.2026-01-18.at.21.07.46.mov
Before (currently in prod/dev)
20260119-1317-08.5602963.mp4
After
Screen.Recording.2026-01-25.at.09.35.24.mov
Before
Screen.Recording.2026-01-25.at.08.18.28.mov
After
Screen.Recording.2026-01-25.at.08.18.00.mov
Before
Screen.Recording.2026-01-25.at.08.43.09.mov
After
Screen.Recording.2026-01-25.at.08.44.14.mov
Before
Screen.Recording.2026-01-25.at.08.18.28.mov
After
Screen.Recording.2026-01-25.at.08.18.00.mov
Screen.Recording.2026-01-30.at.09.55.25.mov
How to Test
https://fair-dev-spatialnode.vercel.app/