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

Tab Improvements #128

Merged
merged 26 commits into from
Mar 21, 2022
Merged

Tab Improvements #128

merged 26 commits into from
Mar 21, 2022

Conversation

austincondiff
Copy link
Collaborator

@austincondiff austincondiff commented Mar 20, 2022

Screen.Recording.2022-03-20.at.5.59.27.PM.mp4

Replicating aesthetic and behavior of standard tabs. Removed wrapper component so that code is more straightforward.

Incorporated changes made by @viktorstrate in #126.

austincondiff and others added 21 commits March 19, 2022 16:54
* Allow opening of `public.item` instead of `public.source-code`

* Handle errors
* Cleaned up the typography for Welcome Screen

* Added support for more code editor themes

* Revert "Added support for more code editor themes"

This reverts commit 8139217.

* Revert "Cleaned up the typography for Welcome Screen"

This reverts commit 78b748b.

* Update the error message to be more informative when trying to open files that are not code related

* Added language translations for editor error and support for the Afrikaans Language

* Fixed localization changes in Russian and Belarusian

* Updated the editor error string to be short and sweet

* Update to editor error for file types.
…component so that code is more straightforward.
Conflicts:
	CodeEdit.xcodeproj/project.pbxproj
	CodeEdit/AppDelegate.swift
	CodeEdit/Documents/WorkspaceDocument.swift
	CodeEdit/Rows/FileTabRow.swift
	CodeEdit/SideBar/SideBarItem.swift
	CodeEdit/TabBar/TabBar.swift
	CodeEdit/TabBar/TabBarItem.swift
Copy link
Member

@pkasila pkasila left a comment

Choose a reason for hiding this comment

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

Looks great 👍

@austincondiff austincondiff merged commit c07955c into main Mar 21, 2022
@austincondiff austincondiff deleted the tab-style branch April 4, 2022 01:52
xinix909 pushed a commit to xinix909/CodeTransfer that referenced this pull request Sep 7, 2024
<!--- IMPORTANT: If this PR addresses multiple unrelated issues, it will
be closed until separated. -->

### Description

1. Add 20 pixels of inset to the left of the ruler and 8 pixels of inset
to the right of the ruler. I tried to match it as close as possible to
Xcode.
2. Set the background color of the highlight ruler line to the same
color as the highlighted line in the textView.

### Related Issues

<!--- REQUIRED: Tag all related issues (e.g. * CodeEditApp#123) -->
<!--- If this PR resolves the issue please specify (e.g. * closes CodeEditApp#123)
-->
<!--- If this PR addresses multiple issues, these issues must be related
to one other -->

* closes CodeEditApp#156
* closes CodeEditApp#128

### Checklist

<!--- Add things that are not yet implemented above -->

- [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

<!--- REQUIRED: if issue is UI related -->

<img width="765" alt="Screenshot 2023-03-16 at 19 05 59"
src="https://user-images.githubusercontent.com/82230675/225717006-40530a7b-7864-404b-8d14-5307f6f83bdb.png">


<!--- IMPORTANT: Fill out all required fields. Otherwise we might close
this PR temporarily -->
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.

5 participants