Skip to content

feat: add polished icons for no content message#2033

Merged
AlexAndBear merged 4 commits intomainfrom
issues/2006
Mar 3, 2026
Merged

feat: add polished icons for no content message#2033
AlexAndBear merged 4 commits intomainfrom
issues/2006

Conversation

@AlexAndBear
Copy link
Member

Description

image

Related Issue

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)

@AlexAndBear AlexAndBear requested a review from kulmann February 24, 2026 09:36
@AlexAndBear AlexAndBear marked this pull request as draft February 24, 2026 10:55
@AlexAndBear
Copy link
Member Author

On hold until the icons and messages are delivered for

  • spaces (files / admin settings)
  • users (admin settings)
  • groups (admin settings)

@AlexAndBear AlexAndBear marked this pull request as ready for review February 26, 2026 10:48
Copy link
Member

@JammingBen JammingBen left a comment

Choose a reason for hiding this comment

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

Love the new images! But I noticed that they are not getting centered vertically anymore. They always are a bit below the vertical center line.

# Conflicts:
#	packages/web-app-admin-settings/src/views/Groups.vue
#	packages/web-app-admin-settings/tests/unit/views/__snapshots__/Spaces.spec.ts.snap
#	packages/web-app-admin-settings/tests/unit/views/__snapshots__/Users.spec.ts.snap
@AlexAndBear
Copy link
Member Author

Love the new images! But I noticed that they are not getting centered vertically anymore. They always are a bit below the vertical center line.

Fixed

Copy link
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.

Love it 😍

@AlexAndBear AlexAndBear merged commit da2b1c2 into main Mar 3, 2026
28 checks passed
@AlexAndBear AlexAndBear deleted the issues/2006 branch March 3, 2026 13:26
@openclouders openclouders mentioned this pull request Mar 3, 2026
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Files app pages: empty states

3 participants