From 29a1c786a3bdcbff681695653217600b541c661b Mon Sep 17 00:00:00 2001 From: 0ko <0ko@noreply.codeberg.org> Date: Tue, 9 Jul 2024 23:38:53 +0500 Subject: [PATCH] ui(settings): make user privacy settings more clear --- options/locale/locale_en-US.ini | 4 +-- release-notes/9.0.0/4439.md | 1 + templates/user/settings/profile.tmpl | 6 ++-- .../integration/user_profile_activity_test.go | 34 ++++++++++++++----- 4 files changed, 32 insertions(+), 13 deletions(-) create mode 100644 release-notes/9.0.0/4439.md diff --git a/options/locale/locale_en-US.ini b/options/locale/locale_en-US.ini index 1d8da735cf..8cd02ee998 100644 --- a/options/locale/locale_en-US.ini +++ b/options/locale/locale_en-US.ini @@ -767,7 +767,7 @@ comment_type_group_issue_ref = Issue reference saved_successfully = Your settings were saved successfully. privacy = Privacy keep_activity_private = Hide activity from profile page -keep_activity_private_popup = Your activity will only be visible to you and the instance admins +keep_activity_private.description = Your public activity will only be visible to you and the instance administrators. lookup_avatar_by_mail = Lookup avatar by email address federated_avatar_lookup = Federated avatar lookup @@ -819,7 +819,7 @@ add_email_success = The new email address has been added. email_preference_set_success = Email preference has been set successfully. add_openid_success = The new OpenID address has been added. keep_email_private = Hide email address -keep_email_private_popup = This will hide your email address from your profile, as well as when you make a pull request or edit a file using the web interface. Pushed commits will not be modified. Use %s in commits to associate them with your account. +keep_email_private_popup = This will hide your email address from your profile. It will no longer be the default for commits made via the web interface, like file uploads and edits, and will not be used for merge commits. Instead a special address %s can be used to associate commits with your account. Note that changing this option will not affect existing commits. openid_desc = OpenID lets you delegate authentication to an external provider. manage_ssh_keys = Manage SSH keys diff --git a/release-notes/9.0.0/4439.md b/release-notes/9.0.0/4439.md new file mode 100644 index 0000000000..60b9539a64 --- /dev/null +++ b/release-notes/9.0.0/4439.md @@ -0,0 +1 @@ +Make descriptions of user privacy settings more visible and clear diff --git a/templates/user/settings/profile.tmpl b/templates/user/settings/profile.tmpl index cd6efbfc1c..a092a42032 100644 --- a/templates/user/settings/profile.tmpl +++ b/templates/user/settings/profile.tmpl @@ -103,16 +103,18 @@
- +
+ {{ctx.Locale.Tr "settings.keep_email_private_popup" .SignedUser.GetPlaceholderEmail}}
- +
+ {{ctx.Locale.Tr "settings.keep_activity_private.description" (printf "/%s?tab=activity" .SignedUser.Name)}}
diff --git a/tests/integration/user_profile_activity_test.go b/tests/integration/user_profile_activity_test.go index c5485db11b..015eeaef7e 100644 --- a/tests/integration/user_profile_activity_test.go +++ b/tests/integration/user_profile_activity_test.go @@ -24,11 +24,6 @@ func TestUserProfileActivity(t *testing.T) { // Activity availability should be the same for guest and another non-admin user, so this is not tested separately userGuest := emptyTestSession(t) - // The hint may contain "Configure" link with an anchor. Verify that it works. - response := userRegular.MakeRequest(t, NewRequest(t, "GET", "/user/settings"), http.StatusOK) - page := NewHTMLParser(t, response.Body) - assert.True(t, page.Find(".checkbox#keep-activity-private").Length() > 0) - // = Public = // Set activity visibility of user2 to public. This is the default, but won't hurt to set it before testing. @@ -41,9 +36,18 @@ func TestUserProfileActivity(t *testing.T) { // Verify the hint for all types of users: admin, self, guest testUser2ActivityVisibility(t, userAdmin, "This activity is visible to everyone, but as an administrator you can also see interactions in private spaces.", true) - testUser2ActivityVisibility(t, userRegular, "Your activity is visible to everyone, except for interactions in private spaces. Configure.", true) + hintLink := testUser2ActivityVisibility(t, userRegular, "Your activity is visible to everyone, except for interactions in private spaces. Configure.", true) testUser2ActivityVisibility(t, userGuest, "", true) + // When viewing own profile, the user is offered to configure activity visibility. Verify that the link is correct and works, also check that it links back to the activity tab. + linkCorrect := assert.EqualValues(t, "/user/settings#keep-activity-private", hintLink) + if linkCorrect { + page := NewHTMLParser(t, userRegular.MakeRequest(t, NewRequest(t, "GET", hintLink), http.StatusOK).Body) + activityLink, exists := page.Find(".field:has(.checkbox#keep-activity-private) .help a").Attr("href") + assert.True(t, exists) + assert.EqualValues(t, "/user2?tab=activity", activityLink) + } + // = Private = // Set activity visibility of user2 to private @@ -56,8 +60,11 @@ func TestUserProfileActivity(t *testing.T) { // Verify the hint for all types of users: admin, self, guest testUser2ActivityVisibility(t, userAdmin, "This activity is visible to you because you're an administrator, but the user wants it to remain private.", true) - testUser2ActivityVisibility(t, userRegular, "Your activity is only visible to you and the instance administrators. Configure.", true) + hintLink = testUser2ActivityVisibility(t, userRegular, "Your activity is only visible to you and the instance administrators. Configure.", true) testUser2ActivityVisibility(t, userGuest, "This user has disabled the public visibility of the activity.", false) + + // Verify that Configure link is correct + assert.EqualValues(t, "/user/settings#keep-activity-private", hintLink) }) } @@ -72,11 +79,14 @@ func testChangeUserActivityVisibility(t *testing.T, session *TestSession, newSta } // testUser2ActivityVisibility checks visibility of UI elements on /?tab=activity -func testUser2ActivityVisibility(t *testing.T, session *TestSession, hint string, availability bool) { +// It also returns the account visibility link if it is present on the page. +func testUser2ActivityVisibility(t *testing.T, session *TestSession, hint string, availability bool) string { + t.Helper() response := session.MakeRequest(t, NewRequest(t, "GET", "/user2?tab=activity"), http.StatusOK) page := NewHTMLParser(t, response.Body) // Check hint visibility and correctness testSelectorEquals(t, page, "#visibility-hint", hint) + hintLink, hintLinkExists := page.Find("#visibility-hint a").Attr("href") // Check that the hint aligns with the actual feed availability assert.EqualValues(t, availability, page.Find("#activity-feed").Length() > 0) @@ -87,10 +97,16 @@ func testUser2ActivityVisibility(t *testing.T, session *TestSession, hint string // Check that the current tab is displayed and is active regardless of it's actual availability // For example, on / it wouldn't be available to guest, but it should be still present on /?tab=activity assert.True(t, page.Find("overflow-menu .active.item[href='/user2?tab=activity']").Length() > 0) + + if hintLinkExists { + return hintLink + } + return "" } -// testUser2ActivityButtonsAvailability check visibility of Public activity tab on main profile page +// testUser2ActivityButtonsAvailability checks visibility of Public activity tab on main profile page func testUser2ActivityButtonsAvailability(t *testing.T, session *TestSession, buttons bool) { + t.Helper() response := session.MakeRequest(t, NewRequest(t, "GET", "/user2"), http.StatusOK) page := NewHTMLParser(t, response.Body) assert.EqualValues(t, buttons, page.Find("overflow-menu .item[href='/user2?tab=activity']").Length() > 0)