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.

Sign up for GitHub

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

Jump to bottom

fix: use folder material design icon #747

Merged
merged 1 commit into from
May 30, 2024
Merged

fix: use folder material design icon #747

merged 1 commit into from
May 30, 2024

Conversation

skjnldsv
Copy link
Member

Before After
localhost_8080_index php_apps_files_files_209_dir=_ (1) localhost_8080_index php_apps_files_files_209_dir=_
localhost_8080_index php_apps_dashboard_ (1) localhost_8080_index php_apps_dashboard_

Fix #748

@skjnldsv skjnldsv self-assigned this May 30, 2024
@skjnldsv skjnldsv added this to the Nextcloud 30 milestone May 30, 2024
@skjnldsv
Copy link
Member Author

/backport to stable28

@skjnldsv
Copy link
Member Author

/backport to stable29

@skjnldsv
Copy link
Member Author

/compile amend /

@skjnldsv skjnldsv mentioned this pull request May 30, 2024
Closed
8 tasks
szaimen
szaimen approved these changes May 30, 2024
View reviewed changes
Copy link

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

LGTM but didnt test

@skjnldsv skjnldsv force-pushed the fix/recommendations-folder branch from e305e26 to 8669a60 Compare May 30, 2024 09:33
@skjnldsv
Copy link
Member Author

/compile amend /

@skjnldsv @nextcloud-command
fix: use folder material design icon
f495cad 
Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@skjnldsv skjnldsv merged commit 69234c7 into master May 30, 2024
21 checks passed
@skjnldsv skjnldsv deleted the fix/recommendations-folder branch May 30, 2024 16:38
@backportbot backportbot

This comment was marked as resolved.

Sign in to view
@skjnldsv

This comment was marked as resolved.

Sign in to view
@backportbot backportbot

This comment was marked as resolved.

Sign in to view
@skjnldsv

This comment was marked as resolved.

Sign in to view
@backportbot backportbot

This comment was marked as resolved.

Sign in to view
@backportbot backportbot

This comment was marked as resolved.

Sign in to view
This was referenced May 30, 2024
Open
Merged
@skjnldsv
Copy link
Member Author

#749
#750

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@susnux susnux susnux left review comments

@Pytal Pytal Pytal approved these changes

@marcoambrosini marcoambrosini marcoambrosini approved these changes

@szaimen szaimen szaimen approved these changes

Assignees

@skjnldsv skjnldsv

Labels
3. to review bug Something isn't working design papercut
Projects
None yet
Milestone
Nextcloud 30
Development

Successfully merging this pull request may close these issues.

[Bug]: Server Colors not in sync
5 participants
@skjnldsv @susnux @Pytal @marcoambrosini @szaimen