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.

Sign up for GitHub

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

Jump to bottom

Desktop notifications show square pic rather than rounded pic #51

Closed
ara4n opened this issue Aug 30, 2015 · 3 comments
Closed

Desktop notifications show square pic rather than rounded pic #51

ara4n opened this issue Aug 30, 2015 · 3 comments
Labels
A-Avatar A-Notifications P3

Comments

@ara4n
Copy link
Member

ara4n commented Aug 30, 2015

No description provided.

@ara4n ara4n added the ui/ux label Aug 30, 2015
@dbkr
Copy link
Member

dbkr commented Sep 7, 2015

HTML5 notifications just get an image that you supply to them, so the only ways to do this would either be to generate a rounded image on a server or do it on the client by doing image manipulation in a canvas element and extracting a picture as a data uri or something: is it worth it?

@ara4n ara4n added P3 ui/ux and removed ui/ux labels Sep 7, 2015
@ara4n ara4n modified the milestone: Ragnarok Nov 29, 2015
@ara4n ara4n removed this from the Ragnarok milestone Apr 13, 2016
dtygel pushed a commit to coletivoEITA/riot-web that referenced this issue May 19, 2017
@MTRNord
Merge pull request element-hq#51 from RiotTranslate/weblate-riot-web-…
71429c4 
…riot-web

Update from Weblate.
@turt2live
Copy link
Member

fwiw, round pictures would appear weird to me. Not many chat applications do this, so it could be jarring.

@jryans jryans removed the Z-UI/UX label Mar 9, 2021
@MadLittleMods
Copy link
Contributor

I think we're good to close this. Native platforms do their own things. Windows toast notifications are square and going against the native behavior would be weird for that platform.

@linuxtim linuxtim mentioned this issue May 11, 2024
Closed
t3chguy added a commit that referenced this issue Oct 17, 2024
@RiotRobot @dbkr @t3chguy
[create-pull-request] automated change (#51)
51a5cf6 
Co-authored-by: dbkr <dbkr@users.noreply.github.com>
Co-authored-by: Michael Telatynski <7t3chguy@gmail.com>
JohnSimonsen added a commit to verji/element-web-v2 that referenced this issue Dec 20, 2024
@JohnSimonsen
Merge pull request element-hq#51 from verji/rm/1955-Hierarchy-UX-1
8dc0c21 
Rm/1955 hierarchy ux 1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-Avatar A-Notifications P3
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@jryans @MadLittleMods @dbkr @turt2live @ara4n