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

[stable25] fix(settings): browser freeze when picking photos directory #2061

Merged
merged 3 commits into from
Sep 26, 2023

Conversation

st3iny
Copy link
Member

@st3iny st3iny commented Sep 25, 2023

This PR fixes a browser freeze caused by nextcloud/nextcloud-focus-trap#2.

Changes by this PR to make it work:

  1. Update @nextcloud/dialogs to 4.2.1 to fix the underlying bug.
  2. Refactor style imports: import '@nextcloud/dialogs/dist/index.css'
  3. Start directory picker at root (/) to prevent weird file picker behavior (due to a potentially missing /Photos folder). Note: The old picker automatically switched to the root directory if the initial directory does not exist. The new one just loads forever.

@st3iny st3iny added bug Something isn't working 3. to review Waiting for reviews labels Sep 25, 2023
@st3iny st3iny added this to the Nextcloud 25.0.13 milestone Sep 25, 2023
@st3iny st3iny requested review from artonge and skjnldsv September 25, 2023 12:14
@st3iny st3iny self-assigned this Sep 25, 2023
@artonge
Copy link
Collaborator

artonge commented Sep 25, 2023

/compile amend /

@nextcloud-command nextcloud-command force-pushed the fix/settings/folder-picker-freeze branch from 9da49ad to 1574c3a Compare September 25, 2023 12:50
@artonge artonge enabled auto-merge September 25, 2023 13:37
st3iny and others added 2 commits September 26, 2023 13:51
Signed-off-by: Richard Steinmetz <[email protected]>
Signed-off-by: nextcloud-command <[email protected]>
Signed-off-by: Louis Chemineau <[email protected]>
@artonge artonge force-pushed the fix/settings/folder-picker-freeze branch from e94909c to 9211bd2 Compare September 26, 2023 11:51
Signed-off-by: Louis Chemineau <[email protected]>
@artonge artonge force-pushed the fix/settings/folder-picker-freeze branch from 1d79389 to 15be988 Compare September 26, 2023 14:10
@artonge artonge merged commit 54bbbf7 into stable25 Sep 26, 2023
25 checks passed
@artonge artonge deleted the fix/settings/folder-picker-freeze branch September 26, 2023 14:31
@blizzz blizzz mentioned this pull request Oct 9, 2023
1 task
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 bug Something isn't working
Projects
Development

Successfully merging this pull request may close these issues.

2 participants