Skip to content

Conversation

@zubeydecivelek
Copy link
Contributor

@zubeydecivelek zubeydecivelek commented Dec 11, 2025

Only added to UI:
Screenshot 2025-12-11 at 17 01 17

Screenshot 2025-12-11 at 17 02 25

Backend, tests were already implemented.

@zubeydecivelek zubeydecivelek linked an issue Dec 11, 2025 that may be closed by this pull request
as="a"
onClick={(e) => {
e.preventDefault();
if (!loading) {
Copy link
Contributor

Choose a reason for hiding this comment

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

this shouldn't be needed given the disabled below. And why the preventDefault?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I realized my assets watch was not working well... so it's not looks like that at the moment sorry :(
with Button it looks like this and no need topreventDefault:
Screenshot 2025-12-11 at 15 55 22

And if we want to use <a> so it'll look like a text button, than we need preventDefault

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Or maybe we can add it here?
Screenshot 2025-12-11 at 16 27 55

Copy link
Contributor

Choose a reason for hiding this comment

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

What about only the trash bin red icon, to keep it small?

Copy link
Contributor Author

@zubeydecivelek zubeydecivelek Dec 11, 2025

Choose a reason for hiding this comment

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

Screenshot 2025-12-11 at 17 01 17

I hope it's good like this :) And do we need the hint?

Screenshot 2025-12-11 at 17 02 25

Copy link
Contributor

Choose a reason for hiding this comment

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

I like it!

@zzacharo zzacharo merged commit ce3a492 into CERNDocumentServer:master Dec 12, 2025
3 checks passed
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.

CLC Sync: add remove feature

3 participants