Skip to content

SaaS Screenshots: Cut out empty spaces - #3425

Merged
mnocon merged 1 commit into
saas-screenshotsfrom
saas-screenshots-cut-off
Sep 29, 2026
Merged

mnocon merged 1 commit into
saas-screenshotsfrom
saas-screenshots-cut-off

Conversation

@adriendupuis

@adriendupuis adriendupuis commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor
Question Answer
JIRA Ticket
Versions
Edition

Reduce width and empty spaces for #3418

Checklist

  • Text renders correctly
  • Text has been checked with vale
  • Description metadata is up to date
  • Redirects cover removed/moved pages
  • Code samples are working
  • PHP code samples have been fixed with PHP CS fixer
  • Added link to this PR in relevant JIRA ticket or code PR

@github-actions

Copy link
Copy Markdown

Preview of modified files: no change to preview.

@adriendupuis
adriendupuis requested a review from mnocon September 28, 2026 15:04
@adriendupuis

Copy link
Copy Markdown
Contributor Author

Add @mnocon If there is not any text/UI bad cut issue, please merge by yourself ;)

@github-actions

Copy link
Copy Markdown

Summary

Status Count
🔍 Total 80172
🔗 Unique 2925
✅ Successful 1223
⏳ Timeouts 0
🔀 Redirected 13
👻 Excluded 78929
❓ Unknown 0
🚫 Errors 20
⛔ Unsupported 0

Errors per input

Errors in site/administration/admin_panel/corporate_admin_panel/index.html

  • [ERROR] file:///home/runner/work/documentation-developer/documentation-developer/repositories/userdoc-saas/site/customer_management/manage_customers/index.html (at 4829:39) | File not found. Check if file exists and path is correct

Errors in site/administration/back_office/integrated_help/index.html

  • [ERROR] file:///home/runner/work/documentation-developer/documentation-developer/repositories/userdoc-saas/site/getting_started/discover_ui/index.html#disable-help-center (at 4886:30) | Cannot find fragment

Errors in site/customer_management/cp_applications/index.html

  • [ERROR] file:///home/runner/work/documentation-developer/documentation-developer/repositories/userdoc-saas/site/customer_management/company_self_registration/index.html (at 4872:68) | File not found. Check if file exists and path is correct

Errors in site/ibexa_products/cohesivo_saas_guide/index.html

Errors in site/ibexa_products/editions/index.html

  • [ERROR] file:///home/runner/work/documentation-developer/documentation-developer/repositories/userdoc-saas/site/customer_management/manage_customers/index.html (at 5074:14) | File not found. Check if file exists and path is correct

Errors in site/ibexa_products/ibexa_experience/index.html

Errors in site/ibexa_products/ibexa_headless/index.html

Errors in site/infrastructure_and_maintenance/security/reporting_issues/index.html

Errors in site/multisite/translations_management/configure_translations_management/index.html

  • [ERROR] file:///home/runner/work/documentation-developer/documentation-developer/repositories/userdoc-saas/site/content_management/translate_content/index.html#manage-translation-services-and-language-pairs (at 4940:17) | Cannot find fragment

Errors in site/recommendations/raptor_integration/raptor_connector/index.html

Errors in site/recommendations/raptor_integration/raptor_connector_guide/index.html

Redirects per input

Redirects in site/ibexa_products/cohesivo_saas_guide/index.html

Redirects in site/ibexa_products/ibexa_experience/index.html

Redirects in site/ibexa_products/ibexa_headless/index.html

Redirects in site/infrastructure_and_maintenance/security/reporting_issues/index.html

Redirects in site/product_catalog/quable/quable/index.html

Redirects in site/product_catalog/quable/quable_guide/index.html

Redirects in site/recommendations/raptor_integration/raptor_connector/index.html

Redirects in site/recommendations/raptor_integration/raptor_connector_guide/index.html

Full Github Actions output

@mnocon mnocon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@mnocon
mnocon merged commit 03a2884 into saas-screenshots Sep 29, 2026
11 of 12 checks passed
@mnocon
mnocon deleted the saas-screenshots-cut-off branch September 29, 2026 07:12
mnocon pushed a commit that referenced this pull request Sep 29, 2026
mnocon added a commit that referenced this pull request Sep 29, 2026
* Current status

* Reinstate screenshots removed in the necessity-bar pass

The 35 removals in 1d95312 were argued as necessity ("the prose already
says it", "decorative"), but the real driver was the cost of recapturing.
With that cost gone, every removal comes back unless it cannot be shot.

29 references reinstated, from 26 new captures:
- distraction_free_mode.png serves two pages
- admin_panel_segment_groups.png and recommendation_blocks.png were
  already on disk, so those two rows are reference-only re-adds

Not reinstated:
- advanced_settings.png - blocked; the Raptor connector is not configured
  on local, so the settings modal cannot be reached
- the 5 cp_* shots - Customer Portal is out of the product and PR #3416
  deletes the page

Four shots get new alt text, because the original string no longer
matches the picture: switch_layout_window.png, img/object_states.png,
img/user_segments.png, and locations.md's Users shot.

locations.md:62 gets its own capture, content_management_users.png,
rather than reusing admin_panel_users.png - that file belongs to
users_admin_panel.md and carries Admin-panel highlights. New basename,
because autolinks resolves by basename across all of docs/.

The six locations.md shots are framed to differ from each other rather
than shipped as six near-identical full windows. Seven shots carry a new
#FF4713 rectangle; the other 22 are plain, as their originals were.

mkdocs build --strict passes, all 29 references resolve, no new duplicate
basenames, nothing over 300KB.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

* CI fix

* Selfreview

* Screenshots: Cut out empty spaces (#3425)

---------

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-authored-by: Adrien Dupuis <61695653+adriendupuis@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants