Commits
- Commit:
d2e2bebcbe7fa3d2d6f9ee84f479d6d0241da8ca
- From:
- Omar Polo <op@omarpolo.com>
- Date:
bp_select: call exit_minibuffer() when done
Currently it is still fine to access ministate after
exit_minibuffer(). In the future, if we ever support recursive
minibuffers it won't.
So, defer the exit_minibuffer() call and use minibuffer_compl_text()
instead of peeking into ministate.
While here, also add the "Bookmarked" message when it successfully
bookmarked the page.
- Commit:
ba8a119df2b3646ede6415168bd7f56bde6d090a
- From:
- Omar Polo <op@omarpolo.com>
- Date:
fix bookmark_page() failure check
it returns -1/0 not 0/1
- Commit:
65c4966563d067115fc09f5e5c8c43a917b1cf03
- From:
- Omar Polo <op@omarpolo.com>
- Date:
rework the history management
It was a huge pile of spaghetti with useless utility functions and
lots of code reaching into the "data structure" (if you can call it
so) to do stuff.
Instead, provide a "module" with a clean interface and noone reaching
in the insides.
- Commit:
449ea6fe32441d2fd3875c5eee5d0c77800cfa7d
- From:
- Omar Polo <op@omarpolo.com>
- Date:
apply load-url-use-heuristic for command line arguments too
This semplifies a bit the handling. humanify_url() now takes the base
URL which we can then use when not using the heuristics.
Command-line arguments now take an implicit base of <file://$PWD/>
when the heuristics are disabled, so that foo.gmi resolves to the
local file even without <./>.
See github issue #10.
- Commit:
cc71f6cb128ea70d8fd676c4cac46c3f25c0f7a5
- From:
- Omar Polo <op@omarpolo.com>
- Date:
convert ir_select_reply() to iri
- Commit:
4bb6a4fa12277e60cec320dacf7968754b26db7d
- From:
- Omar Polo <op@omarpolo.com>
- Date:
add load-url-use-heuristic knob
This is still WIP and may change before the next release.
It could be worthwile to resolve the URLs in load-url using the
current page as base, so that while browsing <gemini://a.org/b/c>,
using load-url to go to "/" would load <gemini://a.org/> instead of
<file:///>. The new option `load-url-use-heuristic', if unset,
provides exactly this behaviour.
It is still a bit rough to use, as for e.g. loading <d.net> would now
yield <gemini://a.org/b/d.net> instead of <gemini://d.net>.
Will need more work in-tree, but for now add it so we can reason about
it.
See github issue #10
- Commit:
36f94f06b08270ab5d4465aa61ae025c98e24c2c
- From:
- Omar Polo <op@omarpolo.com>
- Date:
use the new iri parser in ir_select_gemini
While here also add some error checking.
- Commit:
bd4a08a74023593128f1f6fac9e0f4a093a8441f
- From:
- Omar Polo <op@omarpolo.com>
- Date:
start to refactor the rendering
Previously each vline (visual line) had a full copy of its associated
string, this changes it so it only slices a part of the parent line.
Reduces significantly the memory usage.
This actually worsen the emojify-link glitch reported by Freezr after
the some recent refactoring in the wrapping code. Not a big deal since
I'm about to restructure the whole rendering bit by bit (hopefully!)
- Commit:
8115fd4a99b30c65da4fa4e88552b7aa36b71165
- From:
- Omar Polo <op@omarpolo.com>
- Date:
reset all the state functions regardless of MB_READ vs. MB_COMPLREAD
must_select otherwise is set when MB_COMPREAD but not cleared on
MB_READ, yielding funny errors. While here, move also the other parts
of the state out of the if (MB_COMPLREAD) to avoid this kind of issues
in the future.
- Commit:
f63b8f7342aefba6b3dac50d6790981987c8faa8
- From:
- Omar Polo <op@omarpolo.com>
- Date:
merge the fs into the ui process
The previous separation between the fs and ui process wasn't that good.
The idea was to have a `ui' process tightly sandboxed, but it was a lie
actually. `ui' was one imsg away from making internet connections and
accessing data on the disk, so it wasn't really limited in (almost) any
way.
Furthermore, having to serialize data to/from the fs proc started to
become not really maneagable.
As a first step to fix this situation, join the fs and ui process.
- Commit:
1faa682136aa29eb489b92dcd0e72d28a777f763
- From:
- Omar Polo <op@omarpolo.com>
- Date:
recompute_completions: use the history too when viewing it
- Commit:
3f0b6911f04edeb3b3db82635d5403e519cd365e
- From:
- Omar Polo <op@omarpolo.com>
- Date:
minibuffer_compl_text: honour the history items
- Commit:
0ab6d481eb9306b6f2e175cf1509842e89b173f8
- From:
- Omar Polo <op@omarpolo.com>
- Date:
don't insert current candidate in eecmd_select
after the recent 'must select' thing there's no need to call
minibuffer_insert_current_candidate from eecmd_select anymore.
- Commit:
4bb17137e88ce277e15da09d133b6a3809d26a8a
- From:
- Omar Polo <op@omarpolo.com>
- Date:
readd forgotte NULL check
- Commit:
7c1d55bff32b17bf9ac2b1a374c9cfb98935d847
- From:
- Omar Polo <op@omarpolo.com>
- Date:
readd "select first completion" when must_select is provided
The "don't automatically select the first completion" was a good
change, but for commands that need a must_select policy it's annoying.
This readds the automatic selection but only for those commands.
It's still better than before the changes because now it's clear when
and what gets automatically selected.