Skip to content

FIX: Fixed panels in linux - #6799

Merged
eblanco-ansys merged 8 commits into
mainfrom
fix/linux-icons-panels
Oct 24, 2025
Merged

FIX: Fixed panels in linux#6799
eblanco-ansys merged 8 commits into
mainfrom
fix/linux-icons-panels

Conversation

@eblanco-ansys

Copy link
Copy Markdown
Contributor

Description

Linux panels need a relative path for the icons in order to work. I added a symlink to the environment icons folder so we don't create a copy of them in PersonalLib folder

Checklist

  • I have tested my changes locally.
  • I have added necessary documentation or updated existing documentation.
  • I have followed the coding style guidelines of this project.
  • I have added appropriate tests (unit, integration, system).
  • I have reviewed my changes before submitting this pull request.
  • I have linked the issue or issues that are solved by the PR if any.
  • I have agreed with the Contributor License Agreement (CLA).

@eblanco-ansys
eblanco-ansys requested a review from a team as a code owner October 22, 2025 10:36
@eblanco-ansys
eblanco-ansys enabled auto-merge (squash) October 22, 2025 10:37
@eblanco-ansys eblanco-ansys self-assigned this Oct 22, 2025
@github-actions github-actions Bot added the bug Something isn't working label Oct 22, 2025
@codecov

codecov Bot commented Oct 22, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.32%. Comparing base (f5a2b8c) to head (8f24d46).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6799      +/-   ##
==========================================
+ Coverage   79.15%   83.32%   +4.16%     
==========================================
  Files         246      246              
  Lines       77423    77423              
==========================================
+ Hits        61284    64510    +3226     
+ Misses      16139    12913    -3226     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@SMoraisAnsys SMoraisAnsys left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, I left a minor comment ?

Comment thread src/ansys/aedt/core/extensions/customize_automation_tab.py
@eblanco-ansys

Copy link
Copy Markdown
Contributor Author

LGTM, I left a minor comment ?

I created issue #6813 to work on it for next release. I think we can merge this one

@gmalinve gmalinve 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.

LGTM.

@eblanco-ansys
eblanco-ansys merged commit a25dafd into main Oct 24, 2025
39 checks passed
@eblanco-ansys
eblanco-ansys deleted the fix/linux-icons-panels branch October 24, 2025 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants