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

initial post reply view #107

Merged
merged 3 commits into from
Jun 25, 2024
Merged

initial post reply view #107

merged 3 commits into from
Jun 25, 2024

Conversation

jb55
Copy link
Contributor

@jb55 jb55 commented Jun 21, 2024

i spent way too much time implementing the line drawing

Xnapper-2024-06-21-14 02 59

Signed-off-by: William Casarin <[email protected]>
@jb55
Copy link
Contributor Author

jb55 commented Jun 21, 2024

@robagreda got it pixel perfect this time!

Signed-off-by: William Casarin <[email protected]>
@jb55 jb55 requested a review from kernelkind June 21, 2024 21:20
@kernelkind
Copy link
Member

Looks good! But it needs a ScrollArea because if the note you're replying to is larger than the screen then you can't get to the post box

Suggested-by: kernelkind <[email protected]>
Signed-off-by: William Casarin <[email protected]>
@jb55
Copy link
Contributor Author

jb55 commented Jun 24, 2024 via email

@robagreda
Copy link

This looks great @jb55 should we add a placeholder saying like: type your reply here... something like that?

@jb55
Copy link
Contributor Author

jb55 commented Jun 25, 2024 via email

@jb55 jb55 merged commit a685686 into master Jun 25, 2024
8 checks passed
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