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

Full Quick Look support #1807

Closed

Conversation

LeonardoLarranaga
Copy link
Contributor

@LeonardoLarranaga LeonardoLarranaga commented Jul 11, 2024

Description

This pull request brings full Quick Look support. This means that now, any type of file compatible with Quick Look can be viewed within CodeEdit (such as videos).

Related Issues

Checklist

  • I read and understood the contributing guide as well as the code of conduct
  • The issues this PR addresses are related to each other
  • My changes generate no new warnings
  • My code builds and runs on my machine
  • My changes are all related to the related issue above
  • I documented my code

Screenshots

Videos inside CodeEdit

Menu

Quick Look Menu

Video

Screen.Recording.2024-07-10.at.23.04.25.mov

Update 1

Opened in code editor (left). Opened with Quick Look (right).
Icon when opening with Quick Look.

Update 2

Open As -> Source Code menu item is now enabled. When having a file opened with Quick Look, clicking this option will opened the file in the default CodeEdit editor.

Screen.Recording.2024-07-10.at.23.50.55.mov

@LeonardoLarranaga LeonardoLarranaga changed the title Quick look support Full Quick Look support Jul 11, 2024
Leonardo Larrañaga added 4 commits July 10, 2024 23:24
#Conflicts:
#	CodeEdit/Features/Editor/TabBar/Tabs/Tab/EditorTabView.swift
…pened with Quick Look, clicking this option will opened the default CodeEdit editor.
Copy link
Collaborator

@thecoolwinter thecoolwinter left a comment

Choose a reason for hiding this comment

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

This is looking good, just one small thought on the view rename. I like the move to the menu items being in the quick look controller 👍

Also fix tests, looks like it's having trouble finding a file.

CodeEdit/Features/Editor/Views/QuickLookFileView.swift Outdated Show resolved Hide resolved
@matthijseikelenboom
Copy link
Contributor

There are merge conflicts with main

@LeonardoLarranaga
Copy link
Contributor Author

LeonardoLarranaga commented Jul 15, 2024

[SwiftLint / SwiftLint: CodeEdit/Features/NavigatorArea/ProjectNavigator/OutlineView/ProjectNavigatorViewController.swift#L403](https://github.com/CodeEditApp/CodeEdit/pull/1807/files#annotation_23939490704) File should contain 400 lines or less: currently contains 403 (file_length)

I don't get it, in my pull request I didn't modified that file and it's 396 lines long, waiting for issue to be resolved.

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.

✨ Quick Look support
3 participants