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

Migrate settings menu icons of contacts to material design icons #2807

Merged
merged 1 commit into from
Jul 15, 2022

Conversation

GretaD
Copy link
Contributor

@GretaD GretaD commented Jun 29, 2022

ref nextcloud/groupware#38
before
beforesettings
after
aftersettings
fixes #2777

@GretaD GretaD added the 2. developing Work in progress label Jun 29, 2022
@codecov
Copy link

codecov bot commented Jun 29, 2022

Codecov Report

Merging #2807 (fba8d00) into main (9d1a7cf) will increase coverage by 50.49%.
The diff coverage is n/a.

❗ Current head fba8d00 differs from pull request most recent head 252a525. Consider uploading reports for the commit 252a525 to get more accurate results

@@              Coverage Diff              @@
##               main    #2807       +/-   ##
=============================================
+ Coverage     17.98%   68.47%   +50.49%     
  Complexity      253      253               
=============================================
  Files           101       22       -79     
  Lines          3231      717     -2514     
  Branches        449        0      -449     
=============================================
- Hits            581      491       -90     
+ Misses         2456      226     -2230     
+ Partials        194        0      -194     
Impacted Files Coverage Δ
...nts/AppNavigation/Settings/SettingsAddressbook.vue
.../AppNavigation/Settings/SettingsNewAddressbook.vue
...ts/AppNavigation/Settings/SettingsSortContacts.vue
...omponents/CircleDetails/CirclePasswordSettings.vue
src/components/ContactDetails.vue
src/components/ContactsList.vue
src/components/Properties/PropertyActions.vue
src/components/Properties/PropertyTitle.vue
src/components/ProcessingScreen.vue
src/services/checks/missingFN.js
... and 69 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9d1a7cf...252a525. Read the comment docs.

@GretaD GretaD marked this pull request as ready for review July 4, 2022 15:16
@@ -52,7 +52,7 @@
:disabled="loading.length > 0"
:placeholder="t('contacts', 'Unique password …')"
type="text"
@keyup.enter="saveUniquePassword" />
Copy link
Contributor Author

Choose a reason for hiding this comment

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

phpstorm insist to remove this /

@GretaD GretaD added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Jul 5, 2022
@szaimen
Copy link
Contributor

szaimen commented Jul 6, 2022

@GretaD @st3iny Maybe just as a small heads up: this is how the settings look currently in non-broken form:
image

@ChristophWurst
Copy link
Member

/backport to stable4.2

Copy link
Member

@st3iny st3iny left a comment

Choose a reason for hiding this comment

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

I fixed the fetch-social-avatar checkbox. Otherwise, the PR was good as is.

How it looks now

I think we came pretty close to the screenshot from before.

ksnip_20220715-143254

Signed-off-by: greta <gretadoci@gmail.com>
Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
@st3iny st3iny force-pushed the migrate/settings-contacts branch from 293ab26 to 252a525 Compare July 15, 2022 12:34
@st3iny st3iny added 4. to release Ready to be released and/or waiting for tests to finish bug Something isn't working design Related to the design regression Regression of a previous working feature and removed 3. to review Waiting for reviews labels Jul 15, 2022
@st3iny st3iny merged commit 34964de into main Jul 15, 2022
@st3iny st3iny deleted the migrate/settings-contacts branch July 15, 2022 12:37
@szaimen
Copy link
Contributor

szaimen commented Jul 15, 2022

yes, looks good! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish bug Something isn't working design Related to the design regression Regression of a previous working feature
Projects
Development

Successfully merging this pull request may close these issues.

Huge design regressions on server master
4 participants