blinksh / blink

Blink Mobile Shell for iOS (Mosh based)
https://blink.sh
GNU General Public License v3.0
6.15k stars 575 forks source link

Snippets #1755

Closed carloscabanero closed 1 year ago

carloscabanero commented 1 year ago
### Snippets TODO
- [x] Rename flow
- [x] Delete flow
- [x] Create snippet flow
- [x] Final look and feel (themes)
- [x] Github source
- [x] User may want to force a refresh on snippets locations
- [x] Fixes for small devices
- [x] Initial Snips
- [x] Documentation
- [x] Snippets is Blink+ feature
- [x] Migration for shortcuts
- [x] Comments
- [x] Support iCloud as a Snippets location
- [x] rename snippets folders to snips
- [x] Support multi-line output
- [x] Open Snips locations in Files for editing
- [ ] Release
- [ ] History Snippets
- [ ] Indexes for other non-basic commands
- [ ] Sharing Snippets
- [ ] Snippets contexts? blink/code/remote shell
- [ ] Mark snippets as Dangerous
- [ ] File Extensions
- [ ] Passwords
- [ ] Messenger
- [ ] Add multiple locations. Add another Index and control the Ignore file
carloscabanero commented 1 year ago

@yury For the Create snippet flow, it may be interesting to make more obvious the snippet will be created with certain parameters. Something like:

Screenshot 2023-06-22 at 8 52 29 AM
carloscabanero commented 1 year ago

Few tweaks now that we have multiple Snippets Locations working:

UI tweaks

UX tweaks

Documentation

Other notes

carloscabanero commented 1 year ago

Crazy idea but maybe save on GitHub could generate a PR request?

yury commented 1 year ago

We have issue with NSAttributedString and \t and \\.

Original content len 65: -H \'Content-Type: application/json\' \\\n\t-d \'@${local_file}.json\'

After calling Highlight.text(...) resulting attributed string has different length 62: -H \'Content-Type: application/json\' \\ -d \'@${local_file}.json\'

temporary fix 3dc67fa

yury commented 1 year ago

Another issue with Tap to close gesture. It triggers when you tap/click on text area itself.

carloscabanero commented 1 year ago

We have issue with NSAttributedString and \t and \\.

Original content len 65: -H \'Content-Type: application/json\' \\\n\t-d \'@${local_file}.json\'

After calling Highlight.text(...) resulting attributed string has different length 62: -H \'Content-Type: application/json\' \\ -d \'@${local_file}.json\'

temporary fix 3dc67fa

I think this is looking good disabled. Less distracting, so closing there.

carloscabanero commented 1 year ago
carloscabanero commented 1 year ago