CodeEditApp
🐞 Terminal Rendering Defect
Description
- The bottom line seems to extend further than it should.
- Vertical alignment of characters is slightly off in comparison to Terminal.app
- Resizing window causes artifacts
To Reproduce
n/a
Expected behavior
Version information
CodeEdit: 1.0 Hash: c947d4065290862e415688c062af1788919eaf1a MacOS: 12.3.1 (21E258) Xcode: 13.3.1 (13E500a)
Additional context
n/a
@austincondiff could you provide a more in-detail description and what you did to get this error? That would help me fix it.
Simply open the integrated terminal. If you have any kind of background to your line like in powerline, you will see some stretching at the bottom.
@austincondiff I was looking inside the source files for the TerminalEmulator but couldn't seem to find lines of code that seemed related to this bug.
@SwiftUILabs : UI Bugs could be tricky at all time. I will take a glimpse of it when I am free.
@austincondiff How do you open the terminal? I don't see it anywhere in the UI.
@jevonmao To open the terminal click the rectangular symbol at the bottom right of the window.
@austincondiff I think he didn't open the full-featured UI by opening a file and not a folder. @jevonmao Unfortunately I couldn't make a tutorial, my Xcode is updating. Instead, I added step-by-step instructions below.
- Build the project.
- If you have turned off the welcome view in CodeEdit settings, then turn it on.
- Once you see the welcome view (if you don't, rebuild the project), click on "open an existing file or folder."
- You should see a popup Finder window for you to select a file. Don't select a file, select a folder. For example, if you were working on an app in Xcode prior to this, then you can open that folder.
- You should see the full-featured UI now. If not, please tell me and I will help you.
@SwiftUILabs The issue lies somewhere within SwiftTerm - the library TerminalEmulator relies on. Feel free to open an issue on their repo.
Hello folks,
Would love to track each of these rendering defects in SwiftTerm independently.
I can give some guidance on the first one about “bleeding”. This is sadly a choice, I am also a fan of not bleeding, but as you can see in the history or commits and bugs, it is not a settler problem: some like it one way, some like it the other.
one solution to please everyone is to snap the window size to the character sizes, so there is no room for bleeding or for drawing with the background character. This is not always possible, but it is something that you might want to look into.
There are some band aids that could help, like changing the rendering code to always draw at the bottom; and that could help for console sessions but would trigger the bug on top for full screen apps or apps using colors on top.
As for the other issues, would love to track those separately. I am gone for the next 10 days, but can look at those when I am back
@migueldeicaza Snapping is a great idea even if we snap the contained view instead of the resizable drawer itself.
We would just need to get the height of its container and see how many whole line heights will go into it. Then we would just multiply that number by the line height and set that height to the contained terminal view.
About everything else, as mentioned, we are in no hurry and can wait until you have time. Thanks!