Skip to content

change: exhange icons#2385

Merged
kulmann merged 2 commits intomainfrom
exchange-icons
Apr 17, 2026
Merged

change: exhange icons#2385
kulmann merged 2 commits intomainfrom
exchange-icons

Conversation

@AlexAndBear
Copy link
Copy Markdown
Member

Description

Related Issue

  • Fixes <issue_link>

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Types of changes

  • Bugfix
  • Enhancement (a change that doesn't break existing code or deployments)
  • Breaking change (a modification that affects current functionality)
  • Technical debt (addressing code that needs refactoring or improvements)
  • Tests (adding or improving tests)
  • Documentation (updates or additions to documentation)
  • Maintenance (like dependency updates or tooling adjustments)

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates a couple of UI icons (app switcher + Files “Tiles view”) and adds the required design-system SVG assets to support the new icon name.

Changes:

  • Swap the topbar application switcher icon from apps to apps-3.
  • Update the Files “Tiles view” icon to gallery-view-2 and set fillType: 'none' to match the unsuffixed SVG asset.
  • Add apps-3 SVG assets (-fill and -line) to the design-system icon set.

Reviewed changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File Description
packages/web-runtime/src/components/Topbar/ApplicationsMenu.vue Switches the app switcher button icon to apps-3.
packages/web-app-files/src/composables/extensions/useFolderViews.ts Updates the “Tiles view” icon to gallery-view-2 and uses fillType: 'none'.
packages/design-system/src/assets/icons/apps-3-line.svg Adds the apps-3 line-variant icon asset.
packages/design-system/src/assets/icons/apps-3-fill.svg Adds the apps-3 fill-variant icon asset.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tbsbdr
Copy link
Copy Markdown
Member

tbsbdr commented Apr 17, 2026

as a good sniper-pm I added the request

for list view i suggest list-unordered so that there is no mixup with the mobile burger menu

other than that: perfect! thanks

Copy link
Copy Markdown
Member

@tbsbdr tbsbdr left a comment

Choose a reason for hiding this comment

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

looking diamond!

Copy link
Copy Markdown
Member

@kulmann kulmann left a comment

Choose a reason for hiding this comment

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

🥳

@kulmann kulmann merged commit fca8a43 into main Apr 17, 2026
29 checks passed
@kulmann kulmann deleted the exchange-icons branch April 17, 2026 13:07
@github-project-automation github-project-automation bot moved this from Qualification to Done in OpenCloud Team Board Apr 17, 2026
openclouders pushed a commit that referenced this pull request Apr 17, 2026
* change: exhange icons
@openclouders openclouders mentioned this pull request Apr 17, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants