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

[stable27] 40289 fix appearance #40343

Merged
merged 3 commits into from
Sep 12, 2023
Merged

Conversation

nfebe
Copy link
Contributor

@nfebe nfebe commented Sep 7, 2023

Before Currently Without Icons
2023-09-06 12_10_41-Shared with others - Nextcloud – Mozilla Firefox Screenshot from 2023-09-08 00-23-51 Screenshot from 2023-09-08 00-18-11

@nfebe nfebe added design Design, UI, UX, etc. 3. to review Waiting for reviews 27-feedback labels Sep 7, 2023
@nfebe
Copy link
Contributor Author

nfebe commented Sep 7, 2023

/compile

@AndyScherzinger
Copy link
Member

@juliushaertl @fenn-cs nice 👍 Can we left-align the content so we are closer to the mockups and v28 as described in #26691 ?

@AndyScherzinger AndyScherzinger added this to the Nextcloud 27.1.0 milestone Sep 8, 2023
@blizzz blizzz mentioned this pull request Sep 8, 2023
@nfebe nfebe force-pushed the 40289-stable27-fix-appearance branch from 97d1800 to bbe7e6f Compare September 8, 2023 09:57
@nfebe
Copy link
Contributor Author

nfebe commented Sep 8, 2023

@juliushaertl @fenn-cs nice 👍 Can we left-align the content so we are closer to the mockups and v28 as described in #26691 ?

That was the default, option and I even got a screenshot for it earlier, I just forgot to add it here. The reason, I centered it with the icons, is that, I realized it was centered without them on 27... But yeah done.

Screenshot from 2023-09-08 00-12-15

cc: @AndyScherzinger

@nfebe nfebe requested a review from Pytal September 8, 2023 10:13
@nfebe
Copy link
Contributor Author

nfebe commented Sep 8, 2023

/compile

@nfebe nfebe force-pushed the 40289-stable27-fix-appearance branch 2 times, most recently from 8e054a4 to f7fc1e5 Compare September 8, 2023 13:13
@nfebe
Copy link
Contributor Author

nfebe commented Sep 8, 2023

/compile amend /

@nextcloud-command nextcloud-command force-pushed the 40289-stable27-fix-appearance branch from f7fc1e5 to 500cc77 Compare September 8, 2023 13:44
Comment on lines +955 to +961
&:first-of-type {
flex: 10%;
}

&:last-of-type {
flex: 90%
}
Copy link
Member

Choose a reason for hiding this comment

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

Ideally this should be done in the vue components with a proper slot like you did initially :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah @skjnldsv, this changes where already made in the component, but where not back-ported (not sure why), so went this route.

Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Just design feedback, not blocking:

  • The subline alignment of "Create, update, delete, etc." is too far to the left, it should ideally be left-aligned with the "Custom permissions" text as per the mockup. If the subline is too long it should ellipsize after 1 line

- List custom permissions activated in sublime
- Align checkboxes in advanced settings

Resolves : #40289

Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
@nfebe nfebe force-pushed the 40289-stable27-fix-appearance branch from a9cf4ae to 98b54f1 Compare September 11, 2023 12:17
Signed-off-by: fenn-cs <fenn25.fn@gmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nfebe
Copy link
Contributor Author

nfebe commented Sep 11, 2023

Just design feedback, not blocking:

The subline alignment of "Create, update, delete, etc." is too far to the left, it should ideally be left-aligned with the "Custom >permissions" text as per the mockup. If the subline is too long it should ellipsize after 1 line

Screenshot from 2023-09-11 13-25-59

cc: @jancborchardt

@nfebe nfebe force-pushed the 40289-stable27-fix-appearance branch from 98b54f1 to 8113c02 Compare September 11, 2023 12:30
@nfebe
Copy link
Contributor Author

nfebe commented Sep 11, 2023

/compile amend /

Copy link
Member

@jancborchardt jancborchardt left a comment

Choose a reason for hiding this comment

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

Nice! :)

@nfebe
Copy link
Contributor Author

nfebe commented Sep 11, 2023

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@AndyScherzinger AndyScherzinger changed the title 40289 stable27 fix appearance [stable27] 40289 fix appearance Sep 12, 2023
@blizzz blizzz merged commit 8ffe77d into stable27 Sep 12, 2023
@blizzz blizzz deleted the 40289-stable27-fix-appearance branch September 12, 2023 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews 27-feedback design Design, UI, UX, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants