Skip to content

New css, and the ability to change between servers#279

Open
Bertogim wants to merge 11 commits intochylex:masterfrom
Bertogim:master
Open

New css, and the ability to change between servers#279
Bertogim wants to merge 11 commits intochylex:masterfrom
Bertogim:master

Conversation

@Bertogim
Copy link

@Bertogim Bertogim commented Mar 18, 2025

Added:

  • The ability to switch between servers
  • More accurate CSS
  • User chat profile pictures (PFP)

The only thing I’m unsure how to add is server icons because I need to edit more than just the JS and HTML...

image
image

@chylex chylex self-assigned this Mar 18, 2025
@chylex
Copy link
Owner

chylex commented Mar 18, 2025

Thanks, I'll look through it soon, but I have a few notes right now:

  • The viewer cannot have a dependency on a third-party CDN, it needs to work completely offline and have a reasonable set of fallback fonts. The viewer can reference the gg sans font if the user installs it into their operating system, but the font is proprietary so I cannot bundle it, and I don't want to depend on cdnfonts.
  • I don't think including the Discord logo in the viewer is an allowed use of their branding.
  • I don't think links should open in new tabs by default, that way users always have the option to choose whether to left-click to open in the same tab, or middle-click to open in a new tab. Making all links open in new tabs removes the first option.
  • I can look into adding support for saving server icons before I merge that part of the PR.

@Bertogim
Copy link
Author

  • True, i made fonts into local files
  • Fixed (Asked chatgpt for a svg)
  • Reverted, sorry i did not know what i was thinking
  • That would be awesome!

I also removed some console.log i left

@chylex
Copy link
Owner

chylex commented Mar 18, 2025

True, i made fonts into local files

Unfortunately I cannot include the files either, Discord explicitly does not allow anyone else to use the fonts.

Fixed (Asked chatgpt for a svg)

ChatGPT cannot tell what the license of the icon it generated (or possibly stole) is, I will come up with a replacement or draw my own one if I need to.

@Bertogim
Copy link
Author

Bertogim commented Mar 18, 2025

  • I thought the font was just public, removed it
  • So true, we cant know if its stole, replaced it with DM / Direct Messages (You can replace this as you wish)

@chylex
Copy link
Owner

chylex commented Apr 25, 2025

I added code to store server icons, when I have time I'll integrate it into the PR and clean it up.

@Bertogim
Copy link
Author

Bertogim commented Nov 5, 2025

oops i clicked the wrong button

@Bertogim Bertogim reopened this Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants