fix(a11y): add alt text to images missing it

[swe-sanad]

Problem

Several images across the desk frontend had no alt attribute.

Fix

Same pattern as the sibling fix already sent to frappe/crm: icon-next-to-visible-label images (app menu, theme preview) get alt=""; a standalone image with no adjacent label (the attachment preview) gets a real description from the filename already in scope.

Apps.vue no longer exists on develop — its app-menu icon moved into useApps.ts's appMenuItems(), fixed there instead. AttachmentItem.vue also gained video-attachment support since I last looked — kept that as-is and added the alt fix alongside it.

[mergify]

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

• Queue this pull request

[greptile-apps]

Confidence Score: 4/5

The PR appears safe to merge after the non-blocking duplicate provider announcement is addressed.

Provider logos repeat the same accessible text rendered immediately beside them, while the other alt additions preserve useful labeling.

Files Needing Attention: desk/src/components/Settings/EmailProviderIcon.vue

_{Reviews (1): Last reviewed commit: "fix(a11y): add alt text to images missin..." | Re-trigger Greptile}

[greptile-apps]

Provider name announced twice

When serviceName is supplied, the image alt repeats the adjacent provider name, causing screen readers to announce the same text twice. Mark the redundant logo as decorative.

Suggested change

	<img v-if="logoValue" :src="logoValue" :alt="serviceName || 'Email provider'" class="size-4.5" />
	<img v-if="logoValue" :src="logoValue" alt="" class="size-4.5" />

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!