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

Select lines #2443

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Select lines #2443

wants to merge 14 commits into from

Conversation

harshad1
Copy link
Collaborator

This adds an option to select lines which pops up in the long press menu in the text editor. The menu remains open so we can easily copy or replace selections.

@harshad1
Copy link
Collaborator Author

image

@@ -411,8 +411,7 @@ public GsFileBrowserOptions.Options getFilesystemFragmentOptions(GsFileBrowserOp
public void onFsViewerConfig(GsFileBrowserOptions.Options dopt) {
dopt.descModtimeInsteadOfParent = true;
dopt.rootFolder = _appSettings.getNotebookDirectory();
final File fallback = _appSettings.getFolderToLoadByMenuId(_appSettings.getAppStartupFolderMenuId());
final File file = MarkorContextUtils.getValidIntentFile(getIntent(), fallback);
final File file = MarkorContextUtils.getValidIntentFile(getIntent(), dopt.rootFolder);
Copy link
Owner

Choose a reason for hiding this comment

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

Does Markor still startup with i.e. Favorites or storage if you select that in the filebrowser app settings?

@gsantner
Copy link
Owner

gsantner commented Oct 21, 2024

Might I suggest a unicode symbol instead of text? I know already that translating those two words will result in what you would consider a sentence. And you want to have more then just this option listed 😄

}
}
_lastFilter = constraint.toString().toLowerCase().trim();
_filter(_originalList, _filteredList);
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

we don't want to run an async filter inside load folder as it is already async. So we break the internal processing out into a stand-alone function

@harshad1
Copy link
Collaborator Author

harshad1 commented Nov 9, 2024

Changed from text to icon

Screenshot_20241109-103703

@harshad1
Copy link
Collaborator Author

Resolved merge conflict. This is ready to go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants