Skip to content

Basic implementation of the status bar (non interactive) #99

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

Merged
merged 24 commits into from
Mar 20, 2022

Conversation

lukepistrol
Copy link
Member

@lukepistrol lukepistrol commented Mar 19, 2022

This is a basic non-functional status bar with placeholder items. Don't Merge

(Issue #38)
Screen Shot 2022-03-20 at 01 23 37

Screen Shot 2022-03-20 at 14 27 27

Screen Shot 2022-03-20 at 14 27 55

Screen.Recording.2022-03-20.at.01.27.33.mov
Preview.mov

@lukepistrol lukepistrol added the enhancement New feature or request label Mar 19, 2022
@lukepistrol lukepistrol self-assigned this Mar 19, 2022
Copy link
Collaborator

@0xWDG 0xWDG left a comment

Choose a reason for hiding this comment

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

Watch for signing identity

CODE_SIGN_STYLE = Automatic;
COMBINE_HIDPI_IMAGES = YES;
CURRENT_PROJECT_VERSION = 1;
DEVELOPMENT_ASSET_PATHS = "\"CodeEdit/Preview Content\"";
DEVELOPMENT_TEAM = "";
DEVELOPMENT_TEAM = F992DREF6G;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please revert this to empty, if you are done 😊

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh I forgot.. Needed Previews to work 😄

@lukepistrol lukepistrol linked an issue Mar 20, 2022 that may be closed by this pull request
@lukepistrol lukepistrol added the UI This is UI related label Mar 20, 2022
@lukepistrol
Copy link
Member Author

Once GitHub Workflow is bumped to macOS-12 this can be merged for now.

Further implementation needed once features like version control, compiler, … get added

@lukepistrol lukepistrol marked this pull request as ready for review March 20, 2022 15:26
@lukepistrol lukepistrol merged commit c93de28 into CodeEditApp:main Mar 20, 2022
xinix909 pushed a commit to xinix909/CodeTransfer that referenced this pull request Sep 7, 2024
# Description

Tracks indexes that were not given an explicit color and highlights them
as normal text. This fixes problems where certain characters would
change color (eg: they were put in a comment) and then need to change
back, but were skipped.

This also fixes the problem in CodeEditApp#99 where text was not given an explicit
highlight on paste. [Since `STTextView` does not add default attributes
to pasted text by
default](https://github.com/krzyzanowskim/STTextView/blob/5d137731401d12412d567244facf086c325ff95b/Sources/STTextView/STTextView%2BCopyPaste.swift#L26)
(see the `useTypingAttributes: false` in that method call).

Doing this also seems to have an effect on the annoying glitching that
was caused when entering text in an empty line (see the comment screen
recording)

This PR also adds a helper for converting from an `NSRange` to a
`Range<Int>` and gets rid of a bunch of ugly force unwraps that existed
before. It also adds some convenience methods for modifying `IndexSet`s
using indexes from `NSRange` objects.

# Related Issues

- CodeEditApp#99 

# UI Fixes

### Paste Text (Old)


https://user-images.githubusercontent.com/35942988/212186310-ee50dcaa-ebec-4e21-905b-562fc2cdf940.mov

### Paste Text (new)


https://user-images.githubusercontent.com/35942988/212186342-402ceb89-2122-4431-88ac-2363e6452323.mov

### Multi-Line comment (old)


https://user-images.githubusercontent.com/35942988/212186411-dc77744f-270b-4615-80c6-ed02e7c38e80.mov

### Multi-Line comment (new)


https://user-images.githubusercontent.com/35942988/212186437-ba014944-fa6c-4de9-8ad3-c8105a655418.mov

Closes CodeEditApp#99
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request UI This is UI related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ Status bar
2 participants