Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

View: Design Improvements #419

Closed
20 of 21 tasks
Hephi2 opened this issue Jul 18, 2023 · 3 comments · Fixed by #426
Closed
20 of 21 tasks

View: Design Improvements #419

Hephi2 opened this issue Jul 18, 2023 · 3 comments · Fixed by #426
Assignees
Labels
0. Needs triage Pending approval or rejection. This issue is pending approval. enhancement New feature or request

Comments

@Hephi2
Copy link
Contributor

Hephi2 commented Jul 18, 2023

Is your feature request related to a problem? Please describe.

The following aspects were discussed in a design review call from the 18.07.2023:

Edit View Modal

  • The edit view button in the navigation should open the full edit view modal and not only the edit title modal
  • The edit view button in the view page should not be "hidden" in a menu but visible with text next to the title. At the same time this replaces the edit title button that is currently there
    Within the edit view modal:
  • The save button should be sticky
  • The icon to indicate that the columns are movable should be the one with the two lines
  • Additionally there sohuld be arrow buttons to change the columnorder for keyboard accessbility as in the following example:
    image
  • "Meta Data" should be written as "Metadata"

Column Header Menu

  • The "Manage column" buttons should be buttons with an icon and a text in a list
  • For filtering there should be a two-level menu similar to Github or Nextcloud Mail. The first level should present what is shown here and the selected content, e.g.: Operator: Contains. When clicking on that button, this opens the second level. There you can choose between the values. For the search field value he second level provides a text field to enter a custom search string and all the possible magic values

Shares & Sidebar

  • Don't show to much meta info for columns and shares. The "last edit" information is enough and should be displayed as a subline. The rest could be hidden and only shown in a debugging scenario
  • Change "Add Manage table" button text to "Promote to table manager" and use the crown icon as in talk
  • Replace "Add new share" by "Share with accounts or groups"
  • Replace table manager info text in views by "Open main table to adjust table management permissions"
  • Show the table managers inline with the other shares and add an indicator that the person is table manager
  • Integrations tab can go on right of sharing
  • API URL can be in an input field with a copy button (similar to Files settings WebDAV URL)

Other enhancements

  • Replace "Clone view" by "Duplicate view"
  • Creating a new view should also open the full modal not only the field for selecting the emoji and the title
  • Show an indicator for views that have been filtered, that not all entries are visible
  • Extend the template to include the views as well, e.g. for the vacation requests workflow
  • For Metadata column ID use "ID" instead of "Id"
  • For views that are shared with only create permissions, allow users to temporary view their newly created entries
@Hephi2 Hephi2 added enhancement New feature or request 0. Needs triage Pending approval or rejection. This issue is pending approval. labels Jul 18, 2023
@Hephi2 Hephi2 self-assigned this Jul 18, 2023
@Hephi2
Copy link
Contributor Author

Hephi2 commented Jul 20, 2023

It seems like I can't swap the positions of Sharing and Integration since the NcAppSidebar component seems to order the tabs alphabetically by their names...

@juliushaertl
Copy link
Member

juliushaertl commented Jul 20, 2023

Let's maybe open a separate issue if that is expected or if custom order shall be possible in https://github.com/nextcloud/nextcloud-vue

Originating from https://github.com/nextcloud/nextcloud-vue/pull/451/files#diff-43c55b9d7246a23e8b456853911f8e659a2b7d69925d876e4471e96a179c677eR321-R328 but without further context

@Hephi2
Copy link
Contributor Author

Hephi2 commented Jul 20, 2023

I created a seperate issue to be able to close this one since the other enhancements are implemented.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. Needs triage Pending approval or rejection. This issue is pending approval. enhancement New feature or request
Projects
Archived in project
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants