Merge pull request #746 from pixelfed/frontend-ui-refactor

Add alt text to media presenters, fixes #700

public/mix-manifest.json

@@ -2,6 +2,6 @@
     "/js/activity.js": "/js/activity.js?id=7915246c3bc2b7e9770e",
     "/js/app.js": "/js/app.js?id=ea57709b5a7ef8657ff4",
     "/css/app.css": "/css/app.css?id=0d2d3f52adde32658692",
-    "/js/components.js": "/js/components.js?id=baa9a4a5416d4f10cba0",
+    "/js/components.js": "/js/components.js?id=f6de7bd1e3ec76ab599a",
     "/js/timeline.js": "/js/timeline.js?id=83c85c7144756ee9a72e"
 }

resources/assets/js/components/presenter/MixedAlbumPresenter.vue

@@ -18,7 +18,7 @@
 						<source :src="media.url" :type="media.mime">
 					</video>
 
-					<img v-else-if="media.type == 'Image'" slot="img" class="d-block img-fluid w-100" :src="media.url" :alt="media.description">
+					<img v-else-if="media.type == 'Image'" slot="img" class="d-block img-fluid w-100" :src="media.url" :alt="media.description" :title="media.description">
 
 					<p v-else class="text-center p-0 font-weight-bold text-white">Error: Problem rendering preview.</p>
 
@@ -40,7 +40,7 @@
 					<source :src="media.url" :type="media.mime">
 				</video>
 
-				<img v-else-if="media.type == 'Image'" slot="img" class="d-block img-fluid w-100" :src="media.url" :alt="media.description">
+				<img v-else-if="media.type == 'Image'" slot="img" class="d-block img-fluid w-100" :src="media.url" :alt="media.description" :title="media.description">
 
 				<p v-else class="text-center p-0 font-weight-bold text-white">Error: Problem rendering preview.</p>
 

resources/assets/js/components/presenter/PhotoAlbumPresenter.vue

@@ -13,7 +13,7 @@
 				:interval="0"
 			>
 				<b-carousel-slide v-for="(img, index) in status.media_attachments" :key="img.id">
-					<img slot="img" class="d-block img-fluid w-100" :src="img.url" :alt="img.description">
+					<img slot="img" class="d-block img-fluid w-100" :src="img.url" :alt="img.description" :title="img.description">
 				</b-carousel-slide>
 			</b-carousel>
 		</details>
@@ -27,7 +27,7 @@
 			:interval="0"
 		>
 			<b-carousel-slide v-for="(img, index) in status.media_attachments" :key="img.id">
-				<img slot="img" class="d-block img-fluid w-100" :src="img.url" :alt="img.description">
+				<img slot="img" class="d-block img-fluid w-100" :src="img.url" :alt="img.description" :title="img.description">
 			</b-carousel-slide>
 		</b-carousel>
 	</div>

resources/assets/js/components/presenter/PhotoPresenter.vue

@@ -6,13 +6,13 @@
 				<p class="font-weight-light">(click to show)</p>
 			</summary>
 			<a class="max-hide-overflow" :href="status.url" :class="status.media_attachments[0].filter_class">
-				<img class="card-img-top" :src="status.media_attachments[0].url">
+				<img class="card-img-top" :src="status.media_attachments[0].url" :alt="status.media_attachments[0].description" :title="status.media_attachments[0].description">
 			</a>
 		</details>
 	</div>
 	<div v-else>
 		<div :class="status.media_attachments[0].filter_class">
-			<img class="card-img-top" :src="status.media_attachments[0].url">
+			<img class="card-img-top" :src="status.media_attachments[0].url" :alt="status.media_attachments[0].description" :title="status.media_attachments[0].description">
 		</div>
 	</div>
 </template>