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

Screen API #108

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Screen API #108

wants to merge 2 commits into from

Conversation

AMythicDev
Copy link
Owner

@AMythicDev AMythicDev commented Nov 10, 2023

Currently Minus doesn't have any concept of lines. It only cares about the entire data being formatted according to the available number of columns (line number and highlighted search matches if applicable) and the original text data to minus.. Although it is quite simple for anyone to understand, but its not the most elegant and optimized solution.
For example we cannot do some of these effectively:-

  • Format only a selected number of lines
  • Get better quantitative analysis of the incoming text
  • Easily make each line accessible to the end-application

This might be the first stepping stone to fix issues like #106 and other problems. Hence with screens API we introduce a new type Screen which not only brings in lots of performance improvements to the table but also gets a much more ergonomic API for applications to interact with the text data stored inside minus.

This is very much a work in progress so much cannot be described about it but I will keep this updated as the feature furthur evolves

@AMythicDev AMythicDev self-assigned this Nov 10, 2023
@AMythicDev AMythicDev added this to the v5.6.0 milestone Nov 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo ✔️
Development

Successfully merging this pull request may close these issues.

1 participant