-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
✨ Jump Bar Overflow Accordion Effect #241
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
Labels
Comments
xinix909
pushed a commit
to xinix909/CodeTransfer
that referenced
this issue
Sep 7, 2024
This change implements the functionality of <kbd>⌘</kbd> <kbd>/</kbd> for single line comments. It allows you to toggle between commented and uncommented for the line the cursor is currently on when you press <kbd>⌘</kbd> <kbd>/</kbd>. To do so, I implemented a `keyDown` event recognizer, which listens for when the relevant keys are pressed. If <kbd>⌘</kbd> <kbd>/</kbd> is pressed, it calls a method called `commandSlashCalled()`, which decides which toggle is supposed to happen depending on if the line is already commented or not. It also addresses the situation of special cases of languages like HTML, which need a comment at the beginning and end of the line (essentially a `rangeComment`) to comment a single line. ### Related Issues - CodeEditApp#38 This PR accomplishes part of CodeEditApp#38. I talked with some of the project leads (@FastestMolasses) and they said it makes sense to break CodeEditApp#38 into a couple different issues (I.e. single-line vs highlighted chunks, etc). Single-line comment toggling is completed as of this PR but commenting highlighted code still needs to be completed. ### Checklist - [x] I read and understood the [contributing guide](https://github.com/CodeEditApp/CodeEdit/blob/main/CONTRIBUTING.md) as well as the [code of conduct](https://github.com/CodeEditApp/CodeEdit/blob/main/CODE_OF_CONDUCT.md) - [x] The issues this PR addresses are related to each other - [x] My changes generate no new warnings - [x] My code builds and runs on my machine - [x] My changes are all related to the related issue above - [x] I documented my code ### Screenshots <img width="626" alt="Screenshot 2024-04-19 at 5 44 24 PM" src="https://github.com/CodeEditApp/CodeEditSourceEditor/assets/143217945/1200a5c1-d8ea-48a4-9704-70db7aa23fc7"> --------- Co-authored-by: Abe <abe.malla8@gmail.com>
Is anyone currently working on this issue? I'd be happy to take a look at it if not. |
@swiftlysingh assigned to you. Let me know if you have any questions! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
The jump bar was not previously not handling overflow per #182, but was later fixed in #186, which was a quick fix to allow for the jump bar to be scrollable.
Xcode handles it's overflow a little differently with an accordion-like effect where it condenses each item down to an icon if necessary and when items are hovered over they are expanded.
This is a small detail, but we should be handling overflow this way as well.
Screenshots
CodeEdit currently
Screen.Recording.2025-05-05.at.11.05.19.AM.mov
Target behavior
Screen.Recording.2025-05-05.at.11.00.45.AM.mov
The text was updated successfully, but these errors were encountered: