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

"?" - Contextual help overlay looks unbalanced #7331

Closed
tbsbdr opened this issue Jul 26, 2022 · 12 comments · Fixed by #7404
Closed

"?" - Contextual help overlay looks unbalanced #7331

tbsbdr opened this issue Jul 26, 2022 · 12 comments · Fixed by #7404
Assignees
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working

Comments

@tbsbdr
Copy link
Contributor

tbsbdr commented Jul 26, 2022

Steps to reproduce

  1. click on the contextual helper ex. on "Share with people" or "Share via link"

All bold, padding looks unbalanced

Screenshot 000065@2x

padding looks unbalanced

Screenshot 000066@2x

padding and font sizes look unbalanced

Screenshot 000125@2x

Expected behaviour

Screenshot 000069@2x

Actual behaviour

Contextual help overlay looks unbalanced

@tbsbdr tbsbdr added Priority:p3-medium Normal priority Type:Bug Something isn't working labels Jul 26, 2022
@tbsbdr
Copy link
Contributor Author

tbsbdr commented Jul 27, 2022

update: added "Access Details" screenshot

@kulmann kulmann added this to the 2.0.0 General Availability milestone Jul 27, 2022
@AlexAndBear AlexAndBear self-assigned this Jul 28, 2022
@tbsbdr
Copy link
Contributor Author

tbsbdr commented Jul 28, 2022

update: added figma link

@AlexAndBear
Copy link
Contributor

AlexAndBear commented Jul 28, 2022

@tbsbdr
I know some work is required, but could you make simple screens like that
with all contextual helpers you want to have changed?

You can find the existing c-helpers here:
packages/web-app-files/src/helpers/contextualHelpers.js

image

@AlexAndBear
Copy link
Contributor

ODS change is here - owncloud/owncloud-design-system#2271
Now we need to apply the changes in web

@tbsbdr
Copy link
Contributor Author

tbsbdr commented Jul 29, 2022

could you make simple screens

👍 will do. thx for the ODS Change

@tbsbdr
Copy link
Contributor Author

tbsbdr commented Aug 2, 2022

@pmaier1 can you tell if the helper texts are helpful (and correct)? Would appreciate your opinion. The "helpers" appear on click of one of the "?")
Screenshot 000203@2x

1) Helper Share via link

Helper Share via link

2) Helper Share with people

Helper Share with people

3) Helper Access details

(outlook - we could show more details in the future. currently this is mainly used to keep the shares-list clean, i.e. let not so important infos take a backseat here in the Access details)
Helper Access details

4) Helper Indirect links

Helper Indirect links

5) Helper Space Members

Helper Space Members

@AlexAndBear
Copy link
Contributor

@tbsbdr 😍
Can you provide a link to the texts so I can copy them after @pmaier1 gave feedback ?

@AlexAndBear
Copy link
Contributor

Note: Access details is no oc-ct-helper but oc-drop needs to be adjusted with the same style

@pmaier1
Copy link
Contributor

pmaier1 commented Aug 3, 2022

Nice! Some remarks:

  1. There are also indirect user shares. Are those covered as well?
  2. Space managers also have full editor permissions and can manage space properties like description, disable/enable, delete

@tbsbdr
Copy link
Contributor Author

tbsbdr commented Aug 4, 2022

1) Helper Share via link

Texts:

Select who can access links

Only invited people can access
Account and login is required. Only people from the list "Invited people" can access.

Everyone with the link
No login required. Everyone with the link can access. If you share this link with people from the list "Invited people", they need to login so that their individual assigned permissions can take effect. If they are not logged-in, the permissions of the link take effect.

Learn more (https://doc.owncloud.com/webui/next/owncloud_web/web_for_users.html#sharing)

2) Helper Share with people

Texts:

Invite specific people

Enter a name or group to share this item.

Subfolders
If you share a folder, all of its contents and subfolders will be shared as well.

Notification
Invited people will be notified via email or in-app notification.

Incognito
Invited people can not see who else has access.

“via folder”
The “via folder” is shown next to a share, if access has already been given via a parent folder. Click on the “via folder” to edit the share on its parent folder.

Learn more (https://doc.owncloud.com/webui/next/owncloud_web/web_for_users.html#sharing)

3) Helper Access details

(outlook - we could show more details in the future. currently this is mainly used to keep the shares-list clean, i.e. let not so important infos take a backseat here in the Access details)

Texts

  • Name
  • Additional info
  • Access expires ("No" or <Date>)
  • Shared on
  • Invited by
  • Recent view or download of the share ("Never" or <Date>)

Learn more (Will be documented as soon as we extend the infos)

4) Helper Indirect links

Texts:

What are indirect links?

Indirect links are links giving access by a parent folder.

How to edit indirect links
Indirect links can only be edited in their parent folder. Click on the folder icon below the link to navgate to the parent folder.

Learn more (https://doc.owncloud.com/webui/next/owncloud_web/web_for_users.html#sharing)

5) Helper Space Members

Texts:

Add members to this Space

Enter a name to add people or groups as members to this Space.

What members can do
Members can see who else has access to this space and can access all files in this space. Read or write permissions can be set by the member’s role such as “Viewer” or “Editor”.

What Space managers can do
Members with the Manager role can edit all properties and content of a Space, such as adding or removing members, sharing subfolders with non-members, or creating links to share.

Learn more (https://doc.owncloud.com/webui/next/owncloud_web/web_for_users.html#sharing)

Note on "Learn more" links

  • I would already set the links although the docu is not written in detail for all the items.
  • Setting the links should also make users aware that there is a docu
  • Settings the links should give us a push to document the items i.e. give those items prio as they are not self-explanatory via the ui.
  • Links should be updated to more specific anchors as soon as we have them written / documented (currently they only link to #sharing -anchor) 

cc @EParzefall @mmattel please take notice that we have those contextual helpers "?"

@tbsbdr
Copy link
Contributor Author

tbsbdr commented Aug 4, 2022

Note: Access details is no oc-ct-helper but oc-drop needs to be adjusted with the same style

@janackermann yep, noticed that as well. I thought it's a good idea to unify all kinds of "Informational ui" towards the enduser so that they don't need recognise too many different kinds of "informational ui" elements.

@pmaier1 

  1. There are also indirect user shares. Are those covered as well?

added it to the "share with people" helper text

  1. Space managers also have full editor permissions and can manage space properties like description, disable/enable, delete

Updated the text accordingly: texted it more examplarily "..such as..." to communicate implicity that the helper text is not the docu and should not taken as as comprehensive feature description.

@tbsbdr
Copy link
Contributor Author

tbsbdr commented Aug 4, 2022

"Access details" helpers will get tackeled in #7340

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:p3-medium Normal priority Type:Bug Something isn't working
Projects
No open projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants