IEP-1687 Verify all the screenshots in the IDE document and Update

[sigmaaa]

Description

Updated screenshots in the documentation

Fixes # (IEP-1687)

Type of change

Please delete options that are not relevant.

• This change requires a documentation update

How has this been tested?

check the updated doc

Test Configuration:

• ESP-IDF Version:
• OS (Windows,Linux and macOS):

Dependent components impacted by this PR:

• Component 1
• Component 2

Checklist

• PR Self Reviewed
• Applied Code formatting
• Added Documentation
• Added Unit Test
• Verified on all platforms - Windows,Linux and macOS

Summary by CodeRabbit

• Documentation
  • Switched DFU docs to use bundled/local images instead of external URLs.
  • Updated NVS Partition Editor docs to reference bundled/local images instead of external URLs.
  • Added a new "Charts" figure to the Application Size Analysis docs to illustrate memory-usage visualizations.

[coderabbitai]

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

• @coderabbitai resume to resume automatic reviews.
• @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

• ▶️ Resume reviews
• 🔍 Trigger review

📝 Walkthrough

Walkthrough

Replaced external image URLs with local media paths in DFU and NVS Partition Editor docs, and added a new "Charts" figure under Overview in the Application Size Analysis doc.

Changes

Cohort / File(s)	Summary
Documentation image path updates docs/en/additionalfeatures/dfu.rst, docs/en/additionalfeatures/nvspartitioneditor.rst	Replaced external image URLs with local media paths (../../../media/dfu/dfu_1.png, ../../../media/NvsEditor/nvs_1.png, ../../../media/NvsEditor/nvs_2.png, ../../../media/NvsEditor/nvs_3.png). No code or logic changes.
Application Size Analysis docs docs/en/additionalfeatures/application-size-analysis.rst	Inserted a new "Charts" figure (size_charts.png) with center alignment and caption "Application Size Analysis – Charts"; adjusted section layout. No functional changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

• AndriiFilippov
• alirana01

Poem

🐰 I hopped through docs with a nimble beat,

Swapped distant images for folders neat.
Charts now perched tidy, pictures close and bright,
Quietly nesting, safe from the byte. 🥕✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title references the ticket number (IEP-1687) and indicates the change is about verifying and updating screenshots in IDE documentation, which aligns with the actual changes made.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch IEP-1687

📝 Coding Plan

• Generate coding plan for human review comments

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Tip

You can validate your CodeRabbit configuration file in your editor.

If your editor has YAML language server, you can enable auto-completion and validation by adding # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json at the top of your CodeRabbit configuration file.

[kolipakakondal]

Hi @sigmaaa Thanks for the PR. LGTM.

Could you check these areas as well.

https://docs.espressif.com/projects/espressif-ide/en/latest/additionalfeatures/application-size-analysis.html

https://docs.espressif.com/projects/espressif-ide/en/latest/additionalfeatures/install-esp-components.html

https://docs.espressif.com/projects/espressif-ide/en/latest/additionalfeatures/switchlanguage.html#switch-between-languages-in-the-ide

[sigmaaa]

Hi @sigmaaa Thanks for the PR. LGTM.

Could you check these areas as well.

https://docs.espressif.com/projects/espressif-ide/en/latest/additionalfeatures/application-size-analysis.html

https://docs.espressif.com/projects/espressif-ide/en/latest/additionalfeatures/install-esp-components.html

https://docs.espressif.com/projects/espressif-ide/en/latest/additionalfeatures/switchlanguage.html#switch-between-languages-in-the-ide

Hi @kolipakakondal,

Thanks for the review. Addressed in the latest commit

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@docs/en/additionalfeatures/application-size-analysis.rst`:
- Around line 9-10: Update the intro sentence that currently reads "It has two
sections:" to reflect the added Charts section by changing it to "It has three
sections:" so it matches the subsequent bulleted list (the sentence containing
"It has two sections:" in application-size-analysis.rst).

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 731a7497-a26f-4926-b393-e45cde881e0f

📥 Commits

Reviewing files that changed from the base of the PR and between d41cff6 and 01f5e21.

⛔ Files ignored due to path filters (4)

• media/change_language.png is excluded by !**/*.png
• media/size_charts.png is excluded by !**/*.png
• media/sizeanalysis_details.png is excluded by !**/*.png
• media/sizeanalysis_overview.png is excluded by !**/*.png

📒 Files selected for processing (1)

• docs/en/additionalfeatures/application-size-analysis.rst

[AndriiFilippov]

@sigmaaa hi

https://docs.espressif.com/projects/espressif-ide/en/latest/additionalfeatures/appleveltracing.html

there is no example called "app_trace_to_host" anymore.

quite old Debugger illustration:

https://docs.espressif.com/projects/espressif-ide/en/latest/openocddebugging.html
quite old Debugger illustration as well

https://docs.espressif.com/projects/espressif-ide/en/latest/monitoroutput.html
now we have updated "Launch Terminal" shell

[sigmaaa]

Hi @AndriiFilippov,

Thanks for the review. Our documentation mentions that app_trace_to_host was renamed in ESP-IDF version 5.1, so I think this is fine.

I’ve updated the debugging and terminal screenshots. Thanks again!

[kolipakakondal]

LGTM