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

#UI: solve text wrapping in unseal page #27

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

HamedOsama
Copy link

No description provided.

@vercel
Copy link

vercel bot commented Jan 22, 2023

Someone is attempting to deploy a commit to a Personal Account owned by @chronark on Vercel.

@chronark first needs to authorize it.

@HamedOsama
Copy link
Author

#26

@HamedOsama
Copy link
Author

HamedOsama commented Jan 22, 2023

image

After solving the issue.

@@ -74,7 +74,7 @@ export default function Unseal() {
)}
</div>
) : null}
<pre className="px-4 py-3 mt-8 font-mono text-left bg-transparent border rounded border-zinc-600 focus:border-zinc-100/80 focus:ring-0 sm:text-sm text-zinc-100">
<pre className="px-4 py-3 mt-8 font-mono text-left bg-transparent border rounded border-zinc-600 focus:border-zinc-100/80 focus:ring-0 sm:text-sm text-zinc-100 ">
Copy link
Contributor

@chukwumaokere chukwumaokere Jan 23, 2023

Choose a reason for hiding this comment

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

Looks good. but i think you accidentally put a space in here.

Also you probably know this but you should change your api key now that you've posted a picture of it publicly.

@vercel
Copy link

vercel bot commented Jan 23, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
envshare ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 23, 2023 at 11:10AM (UTC)

@chronark
Copy link
Owner

Hmm this doesn't seem to be working for me

image

it splits if I add an empty space, but then the line numbers on the left are no longer correct (which might not be a huge problem actually, but it would be cool if the line numbers stay consistent)

@HamedOsama
Copy link
Author

@chronark I will check and solve it don't worry.

@HamedOsama
Copy link
Author

@chronark Can you check this update?

@chronark
Copy link
Owner

image

@chronark
Copy link
Owner

tbh not sure if the line numbers can be fixed :D

@HamedOsama
Copy link
Author

@chronark I made a new PR can you build it first and try again.

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.

3 participants