Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Aug 28, 2025

Backport of #7401

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot requested review from Antreesy, ShGKme, kra-mo and susnux August 28, 2025 19:37
@backportbot backportbot bot added enhancement New feature or request 3. to review Waiting for reviews labels Aug 28, 2025
@backportbot backportbot bot added this to the 8.29.2 milestone Aug 28, 2025
@AndyScherzinger AndyScherzinger modified the milestones: 8.29.2, 8.30.0 Aug 29, 2025
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
@ShGKme ShGKme force-pushed the backport/7401/stable8 branch from f39f3c8 to 1a08b28 Compare September 1, 2025 17:56
@ShGKme ShGKme marked this pull request as ready for review September 1, 2025 17:56
Escape: !props.mac
? t('Escape') // TRANSLATORS: Escape key on keyboard
: '',
Space: t('Space'), // TRANSLATORS: Space key on keyboard
Copy link
Contributor

Choose a reason for hiding this comment

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

Is comma here not a problem? I remember we just discussed it

Copy link
Contributor

Choose a reason for hiding this comment

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

No, only being the third operand here is a problem.

See updated translations.

sassOptions: {
sourceMapContents: false,
loadPaths: [
includePaths: [
Copy link
Contributor

Choose a reason for hiding this comment

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

This breaks netlifiy as the stylesheets can no longer be found

Copy link
Contributor

Choose a reason for hiding this comment

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

Works locally and on main...

Copy link
Contributor

Choose a reason for hiding this comment

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

Checking

Copy link
Contributor

@ShGKme ShGKme Sep 2, 2025

Choose a reason for hiding this comment

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

Ok, on the main loader is outdated

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
@ShGKme ShGKme force-pushed the backport/7401/stable8 branch from 1a08b28 to e3b0744 Compare September 2, 2025 07:54
@susnux susnux merged commit 276dd43 into stable8 Sep 2, 2025
18 of 19 checks passed
@susnux susnux deleted the backport/7401/stable8 branch September 2, 2025 08:03
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 enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants