BlazeStream#679
Merged
Merged
Conversation
- Clarify the competitive services clause in `docs/services.html` by rephrasing "competing services" to "services that compete with" for improved precision in the terms and conditions. - Remove `docs/services-vdo.html`, indicating a consolidation or deprecation of specific VDO.Ninja-related service documentation. [auto-enhanced]
This commit addresses several issues and improvements related to the display and interaction
of portfolio images within the services documentation.
- Adjust `docs/css/services.css` to ensure portfolio modal images use `object-fit: contain`
and are block elements, preventing distortion and improving layout within the modal.
- Modify `docs/js/services.js` to:
- Prevent click event propagation (`e.stopPropagation`) on portfolio thumbnails to avoid
unintended parent element interactions.
- Introduce `data-full-src` attribute on portfolio `<img>` tags, allowing a dedicated
full-resolution source for the modal while potentially using optimized thumbnails.
- Increase the number of displayed portfolio items per service from 3 to 5.
- Improve `openPortfolioModal` robustness by checking for a valid `src` and forcing
a DOM reflow (`modal.offsetHeight`) to ensure consistent display activation.
These changes collectively improve the visual presentation and user experience of the
service portfolio section, resolving issues related to image scaling and modal interaction.
[auto-enhanced]
- Apply `overflow: hidden` to `.portfolio-image-gallery` to prevent unwanted scrolling and ensure proper layout. - Force `display: block` for `.portfolio-modal-img` to correct image rendering within the portfolio modal. - These adjustments enhance the visual presentation and functionality of images in the services documentation, continuing previous work on the portfolio modal. [auto-enhanced]
Integrates the Blaze.stream live streaming platform as a new supported chat source. - Updates `manifest.json`: - Increments extension version to `3.39.2`. - Adds `https://blaze.stream/*` to `host_permissions`. - Registers `sources/blaze.js` as a content script for `blaze.stream` URLs. - Registers `Blaze.stream` in `docs/js/sites.js` with a description, usage instructions, and a dedicated `blaze.png` icon. - Updates documentation files (`README.md`, `landing.html`, `seo.md`) to list Blaze.stream among supported platforms. - Includes `sources/blaze.js` for platform-specific chat handling and `sources/images/blaze.png` for UI representation. - Note: Chat integration for `blaze.stream` currently functions without pop-out support, suggesting the video should be paused to keep chat open. [auto-enhanced]
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
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.
Uh oh!
There was an error while loading. Please reload this page.