Skip to content

feat(files): add sharing icon in header - #40192

Merged
skjnldsv merged 2 commits into
masterfrom
feat/sharing-icon-bread
Sep 4, 2023
Merged

feat(files): add sharing icon in header #40192
skjnldsv merged 2 commits into
masterfrom
feat/sharing-icon-bread

Conversation

@skjnldsv

@skjnldsvskjnldsv commented Sep 1, 2023

Copy link
Copy Markdown
Member
Not shared
2023-09-01_18-10
Shared by link
2023-09-01_18-10_2
Shared
2023-09-01_18-10_1

@skjnldsvskjnldsv self-assigned this Sep 1, 2023
@skjnldsvskjnldsv changed the title feat(files): add uploaderfeat(files): add sharing icon in header Sep 1, 2023
@skjnldsvskjnldsv mentioned this pull request Sep 1, 2023
26 tasks
Comment threadapps/sharebymail/lib/Capabilities.php Fixed
@skjnldsv
skjnldsvforce-pushed the feat/f2v/uploader branch 2 times, most recently from 28b3070 to 52590a7CompareSeptember 1, 2023 12:35
Base automatically changed from feat/f2v/uploader to masterSeptember 1, 2023 12:54
@skjnldsv
skjnldsvforce-pushed the feat/sharing-icon-bread branch 4 times, most recently from 7f04a0a to 6a91fa3CompareSeptember 1, 2023 14:05
@skjnldsv
skjnldsv requested review from a team, Pytal, artonge, provokateurin and sorbaugh and removed request for a teamSeptember 1, 2023 14:05
@skjnldsvskjnldsv added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Sep 1, 2023
@skjnldsvskjnldsv added this to the Nextcloud 28 milestone Sep 1, 2023
@szaimen

This comment was marked as resolved.

@provokateurin
provokateurin removed their request for review September 1, 2023 15:00
@skjnldsv

skjnldsv commented Sep 1, 2023

Copy link
Copy Markdown
MemberAuthor

@szaimen Feel free to open an issue, but for now this is feature parity.
Please approve so we can move forward :)

EDIT: Unless you can get to an agreement with the design team before it's merged.
But please do not hold back reviews 🙇‍♀️

@tcitworld

This comment was marked as resolved.

@nimishavijay

This comment was marked as resolved.

@skjnldsv

This comment was marked as resolved.

@skjnldsv
skjnldsvforce-pushed the feat/sharing-icon-bread branch from 6a91fa3 to 088a382CompareSeptember 1, 2023 16:12
@skjnldsv

Copy link
Copy Markdown
MemberAuthor

The indicator when "not shared" had an opacity of 0.2 to indicate that it's not currently shared, it's just a button to reach the sharing sidebar.

Here you go, we now have exactly what we had on the old files app.
For any change, please open a dedicated issue, let's not discuss this here, thank you :)

@szaimen

Copy link
Copy Markdown
Contributor

I created #40202

@szaimen

szaimen commented Sep 1, 2023

Copy link
Copy Markdown
Contributor

I tested this and the icon does not seem to directly update after I shared the folder for example. I guess this should be the case though? Apart from that seems to work well so good work!

@skjnldsv

Copy link
Copy Markdown
MemberAuthor

I tested this and the icon does not seem to directly update after I shared the folder for example. I guess this should be the case though? Apart from that seems to work well so good work!

Needs a separate fix. We'll need to do a pass on global events. But right now the Sidebar doesn't update the files store, you are correct :)

Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
@skjnldsv
skjnldsvforce-pushed the feat/sharing-icon-bread branch from 088a382 to 2592568CompareSeptember 1, 2023 23:16
@szaimen

Copy link
Copy Markdown
Contributor

I tested this and the icon does not seem to directly update after I shared the folder for example. I guess this should be the case though? Apart from that seems to work well so good work!

Needs a separate fix. We'll need to do a pass on global events. But right now the Sidebar doesn't update the files store, you are correct :)

Shall I create a follow-up issue then?

@lhsazevedolhsazevedo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Working well for me as well

@szaimen

Copy link
Copy Markdown
Contributor

Shall I create a follow-up issue then?

I've created #40263

@szaimenszaimen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Okay for me then for now

@skjnldsv
skjnldsv merged commit fe692f2 into masterSep 4, 2023
@skjnldsv
skjnldsv deleted the feat/sharing-icon-bread branch September 4, 2023 22:28
@marcoambrosinimarcoambrosini mentioned this pull request Jan 10, 2024
5 tasks
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsfeature: files

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants

@skjnldsv@szaimen@tcitworld@nimishavijay@lhsazevedo@github-advanced-security