Skip to content

Tiles view, replace sort select with less visual obstrutive filter-chip#245

Merged
AlexAndBear merged 1 commit intomainfrom
issues/214-2
Feb 27, 2025
Merged

Tiles view, replace sort select with less visual obstrutive filter-chip#245
AlexAndBear merged 1 commit intomainfrom
issues/214-2

Conversation

@AlexAndBear
Copy link
Copy Markdown
Member

@AlexAndBear AlexAndBear commented Feb 26, 2025

Description

image

Related Issue

How Has This Been Tested?

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

Types of changes

  • New feature (an additional functionality that doesn't break existing code)
  • 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 marked this pull request as ready for review February 26, 2025 12:50
@AlexAndBear
Copy link
Copy Markdown
Member Author

AlexAndBear commented Feb 26, 2025

@tbsbdr is not happy, needs iteration

@tbsbdr code + screenshot updated, feedback plssss ;-)

</svg></button><svg xmlns="http://www.w3.org/2000/svg" width="14" height="10" role="presentation" class="vs__open-indicator">
<path d="M9.211364 7.59931l4.48338-4.867229c.407008-.441854.407008-1.158247 0-1.60046l-.73712-.80023c-.407008-.441854-1.066904-.441854-1.474243 0L7 5.198617 2.51662.33139c-.407008-.441853-1.066904-.441853-1.474243 0l-.737121.80023c-.407008.441854-.407008 1.158248 0 1.600461l4.48338 4.867228L7 10l2.211364-2.40069z"></path>
</svg>
</svg></button><span class="oc-icon oc-icon-s oc-icon-passive"><!----></span>
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

From my other pr, missed to update snapshots, result is correct

@AlexAndBear
Copy link
Copy Markdown
Member Author

AlexAndBear commented Feb 27, 2025

Toby said yes, r2r :)

@tbsbdr
Copy link
Copy Markdown
Member

tbsbdr commented Feb 27, 2025

nice, thank you!

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

LGTM. While you're at it, maybe you can check why sorting by anything other than name or date crashes on the Space overview? Happens on main as well though.

@AlexAndBear
Copy link
Copy Markdown
Member Author

LGTM. While you're at it, maybe you can check why sorting by anything other than name or date crashes on the Space overview? Happens on main as well though.

Already saw that 👍

@AlexAndBear AlexAndBear merged commit 4b28ad6 into main Feb 27, 2025
@AlexAndBear AlexAndBear deleted the issues/214-2 branch February 27, 2025 10:11
@openclouders openclouders mentioned this pull request Mar 17, 2025
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.

Make "Sort by" visually less obstrusive

4 participants