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.

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

Forwarding chat messages #1864

Merged
merged 9 commits into from
Jul 7, 2023
Merged

Forwarding chat messages #1864

merged 9 commits into from
Jul 7, 2023

Conversation

Tenari
Copy link
Contributor

@Tenari Tenari commented Jul 5, 2023

Ticket: RE-157

still left to do:

  • make the spaces version of the ui look better
  • plug in the actual Courier icon
image image

@Tenari Tenari marked this pull request as draft July 5, 2023 22:34
@holium holium deleted a comment from vercel bot Jul 6, 2023
@gdbroman
Copy link
Contributor

gdbroman commented Jul 7, 2023

plug in the actual Courier icon

@Tenari I think it's completely fine to use the png since it doesn't need dynamic colors, and might even be as large in file size, but if you want to try it, you can right-click> copy as SVG here: https://www.figma.com/file/3NMAfjSUcuJ9TasiLwocNt/Realm-2023?type=design&node-id=4998-29880&mode=design&t=4D6jloRkYSiZOPRY-4

@Tenari Tenari marked this pull request as ready for review July 7, 2023 15:09
@holium holium deleted a comment from vercel bot Jul 7, 2023
@Tenari Tenari requested a review from gdbroman July 7, 2023 15:09
lib/design-system/src/blocks/Bubble/Bubble.tsx Outdated Show resolved Hide resolved
Comment on lines +323 to +325
<image
width="24"
height="24"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just create a file called CourierIcon that exports this, no need to shoehorn an image into the SVG component, none of the props will work on this anyway.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I tried to export the icon from figma as SVG it gave me this

also, opacity prop works, as do some others.

lib/design-system/src/general/ShareModal/ShareModal.tsx Outdated Show resolved Hide resolved
app/src/renderer/components/ShareModal/ShareModal.tsx Outdated Show resolved Hide resolved
@holium holium deleted a comment from vercel bot Jul 7, 2023
@Tenari Tenari merged commit c8e5fce into master Jul 7, 2023
@Tenari Tenari deleted the forwarding-chat branch July 7, 2023 21:28
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