Skip to content

Commit f8096ec

Browse files
committed
fix(ImageView): use cursor: pointer for clickable items
Also move the click handler for non-image attachments from the full-width container box to the attachment box with a visible boundary. Signed-off-by: Jonas <jonas@freesources.org>
1 parent 1ac7b79 commit f8096ec

1 file changed

Lines changed: 14 additions & 4 deletions

File tree

src/nodes/ImageView.vue

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,10 @@
2323
<div
2424
v-if="isMediaAttachment"
2525
contenteditable="false"
26-
class="media"
27-
@click="handleAttachmentClick">
28-
<div class="media__wrapper">
26+
class="media">
27+
<div
28+
class="media__wrapper clickable"
29+
@click="handleAttachmentClick">
2930
<img
3031
v-show="loaded"
3132
:src="imageUrl"
@@ -53,7 +54,7 @@
5354
v-show="loaded"
5455
:src="imageUrl"
5556
:alt="alt"
56-
class="image__main"
57+
class="image__main clickable"
5758
@click="handleImageClick"
5859
@load="onLoaded" />
5960
</div>
@@ -552,6 +553,14 @@ export default {
552553
justify-content: left;
553554
padding-bottom: 0;
554555
556+
.clickable {
557+
cursor: pointer;
558+
559+
* {
560+
cursor: pointer;
561+
}
562+
}
563+
555564
.media__wrapper {
556565
display: flex;
557566
border: 2px solid var(--color-border);
@@ -579,6 +588,7 @@ export default {
579588
}
580589
}
581590
}
591+
582592
.buttons {
583593
margin-left: 8px;
584594
}

0 commit comments

Comments
 (0)