Page MenuHomePhorge

ux cleanup. move 'delete account' button to the top
ClosedPublic

Authored by bohlender on Dec 17 2020, 9:47 AM.
Tags
None
Referenced Files
F16102593: D1954.id6022.diff
Tue, Oct 15, 3:29 AM
Unknown Object (File)
Wed, Oct 9, 10:31 AM
Unknown Object (File)
Fri, Oct 4, 4:59 AM
Unknown Object (File)
Sun, Sep 22, 5:20 AM
Unknown Object (File)
Sep 13 2024, 8:55 AM
Unknown Object (File)
Sep 13 2024, 8:54 AM
Unknown Object (File)
Sep 13 2024, 8:54 AM
Unknown Object (File)
Sep 13 2024, 8:51 AM
Subscribers
Restricted Project

Details

Diff Detail

Repository
rK kolab
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bohlender created this revision.
bohlender added a reviewer: Restricted Project.
bohlender added a subscriber: Restricted Project.

I'm not sure. Is it an exception for this one page or this is a general rule that delete button should be on top? E.g. regarding https://bifrost.kolabsystems.com/T375653 What if the button will be in bottom-right corner?

I'm not sure.

The main goal was to move it away from the submit button.
It should neither be on the same line as the submit button, nor in the form to make it clear that this is not an part of the actual form.

It being on the top is in line with the top "add" button being on the top of lists.

What if the button will be in bottom-right corner?

I like it a bit less but it would work on desktop. Less so on mobile UI. There the buttons would be close again.

Is it an exception for this one page or this is a general rule that delete button should be on top? E.g. regarding https://bifrost.kolabsystems.com/T375653

It is intended as a general pattern that would work for user accounts, user domains and so on.

vanmeeuwen subscribed.

Can we, in this case, ensure that it does get applied across the board consistently?

This revision now requires changes to proceed.Dec 23 2020, 11:40 AM
  • delete users from Info instead of list
machniak requested changes to this revision.Feb 2 2021, 1:49 PM

A couple of browser tests need to be modified in tests/Browser/UsersTest.php

src/resources/vue/User/Info.vue
417

.modal('hide') is redundant here.

src/resources/vue/User/List.vue
28

Remove 'colspan' here.

This revision now requires changes to proceed.Feb 2 2021, 1:49 PM
  • remove obsolte colspan and modal('hide')

Browser tests need to be updated.

This revision now requires changes to proceed.Apr 8 2021, 12:45 PM
  • Fix tests
  • Small color improvements
This revision was not accepted when it landed; it landed in state Needs Review.Apr 21 2021, 11:39 AM
This revision was automatically updated to reflect the committed changes.