Skip to content

enh: add data-handler when attached to el - #1371

Merged
max-nextcloud merged 1 commit into
masterfrom
enh/data-handler-when-attached-to-el
Sep 14, 2022
Merged

enh: add data-handler when attached to el#1371
max-nextcloud merged 1 commit into
masterfrom
enh/data-handler-when-attached-to-el

Conversation

@max-nextcloud

Copy link
Copy Markdown
Contributor

Also provide a data-handler attribute to the viewer div when the viewer is attached to a dom element other than the modal.

@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.

Lgtm but didnt test

Comment threadsrc/views/Viewer.vue Outdated
max-nextcloud added a commit to nextcloud/server that referenced this pull request Sep 14, 2022
Based on nextcloud/viewer#1371 .
If we are displaying an audio file in a single page share
provide enough margin to display the speed settings.
Signed-off-by: Max <max@nextcloud.com>
@max-nextcloud
max-nextcloudforce-pushed the enh/data-handler-when-attached-to-el branch from 11d03d2 to b8cd522CompareSeptember 14, 2022 09:14

@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.

works

@szaimenszaimen added this to the Nextcloud 25 milestone Sep 14, 2022
@szaimenszaimen added 3. to review Waiting for reviews enhancement New feature or request labels Sep 14, 2022
@szaimen

Copy link
Copy Markdown
Contributor

/compile amend /

Also provide a data-handler attribute to the viewer div
when the viewer is attached to a dom element other than the modal.
Signed-off-by: Max <max@nextcloud.com>
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@nextcloud-command
nextcloud-commandforce-pushed the enh/data-handler-when-attached-to-el branch from b8cd522 to 488d223CompareSeptember 14, 2022 09:48
@max-nextcloud
max-nextcloud merged commit e51feed into masterSep 14, 2022
@max-nextcloud
max-nextcloud deleted the enh/data-handler-when-attached-to-el branch September 14, 2022 10:03
susnux pushed a commit that referenced this pull request May 16, 2026
…d-to-el
enh: add data-handler when attached to el
susnux pushed a commit that referenced this pull request May 16, 2026
…d-to-el
enh: add data-handler when attached to el
susnux pushed a commit that referenced this pull request May 16, 2026
…d-to-el
enh: add data-handler when attached to el
susnux pushed a commit that referenced this pull request May 16, 2026
…d-to-el
enh: add data-handler when attached to el
susnux pushed a commit that referenced this pull request May 16, 2026
…d-to-el
enh: add data-handler when attached to el
susnux pushed a commit that referenced this pull request May 16, 2026
…d-to-el
enh: add data-handler when attached to el
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsenhancementNew feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@max-nextcloud@szaimen@skjnldsv