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

fix(Question): close question menu after copy question #1851

Merged
merged 1 commit into from
Dec 26, 2023

Conversation

Chartman123
Copy link
Collaborator

This fixes some focus jumping after copying a question by closing the action menu after the click.

Signed-off-by: Christian Hartmann chris-hartmann@gmx.de

Signed-off-by: Christian Hartmann <chris-hartmann@gmx.de>
@Chartman123 Chartman123 added the bug Something isn't working label Dec 24, 2023
@Chartman123 Chartman123 added javascript Javascript related ticket 3. to review Waiting for reviews regression Regression of a previous working feature labels Dec 24, 2023
@Chartman123 Chartman123 added this to the 4.1 milestone Dec 24, 2023
@Chartman123 Chartman123 self-assigned this Dec 24, 2023
@Chartman123 Chartman123 changed the title fix: close question menu after copy question fix(Question): close question menu after copy question Dec 24, 2023
Base automatically changed from fix/actions-button to main December 24, 2023 10:09
@@ -99,7 +99,8 @@
</template>
{{ t('forms', 'Technical name') }}
</NcActionInput>
<NcActionButton @click="onClone">
<NcActionButton :close-after-click="true"
Copy link
Collaborator

Choose a reason for hiding this comment

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

For boolean properties this is not needed and depends on your preference / styleguide as this is the same like:

Suggested change
<NcActionButton :close-after-click="true"
<NcActionButton close-after-click

@susnux susnux merged commit 4a6ba22 into main Dec 26, 2023
33 checks passed
@susnux susnux deleted the fix/close-action-menu-after-copy-question branch December 26, 2023 18:17
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 feature: 📑 form creation javascript Javascript related ticket regression Regression of a previous working feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants