Skip to content

feat: added support symlinks in "list view" mode - #2003

Open
xelavopelk wants to merge 1 commit into
pop-os:masterfrom
xelavopelk:add_symlink_icon
Open

feat: added support symlinks in "list view" mode#2003
xelavopelk wants to merge 1 commit into
pop-os:masterfrom
xelavopelk:add_symlink_icon

Conversation

@xelavopelk

@xelavopelk xelavopelk commented Aug 24, 2026

Copy link
Copy Markdown
  • I have disclosed use of any AI generated code in my commit messages.
    • If you are using an LLM, and do not fully understand the changes it is making to the code base, do not create a PR.
    • In our experience, AI generated code often results in overly complex code that lacks enough context for a proper fix or feature inclusion. This results in considerably longer code reviews. Due to this, AI authored or partially authored PRs may be closed without comment.
  • I understand these changes in full and will be able to respond to review comments.
  • My change is accurately described in the commit message.
  • My contribution is tested and working as described.
  • I have read the Developer Certificate of Origin and certify my contribution under its conditions.

Added support for symlinks for "list view" mode. Details can be found at the following link: issue.

Due to the absence of the "emblem-symbolic-link" icon, the more suitable of the existing ones was chosen - "mail-forward-symbolic".
I did a little refactoring of the "list_view" function, as it seemed too complicated to me. Some of the string generation methods have been moved to the ListViewRowBuilder abstraction. Refactoring is shallow to reduce the size of the PR.
It works in wide list mode.
image

It works in the list mode in an brief mode.
image

It works for the trash.
image

@xelavopelk xelavopelk changed the title feat: added support for symlinks for "list view" mode feat: added support symlinks in "list view" mode Aug 24, 2026
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.

1 participant