Commits
- Commit:
0e7b2e99bcc7b27a70cab7a553746ca3ed98ccac
- From:
- Omar Polo <op@omarpolo.com>
- Date:
rate-limit the update of the download pane
- 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:
27dbcaab4fac08fbb43df9eb5f10781ff7015bb6
- From:
- Omar Polo <op@omarpolo.com>
- Date:
add must_select flag for enter_minibuffer
it only makes sense when entering the minibuffer with completions.
This flag is useful for functions like tag-select where the user types
something but one of the completions *must* be selected.
- Commit:
0a98735841e8b33edf62bd756ad8c6da2929247a
- From:
- Omar Polo <op@omarpolo.com>
- Date:
ring a bell when remotely opening a link + refactor it a bit
move the logic into its own function.
- Commit:
693188cbff678ee0934f5fe5da91365614864888
- From:
- Omar Polo <op@omarpolo.com>
- Date:
ui_require_input: take a fn pointer instead of the protocol number
- Commit:
6d5348f32514004ad33d27ece8dec68ffed9b209
- From:
- Omar Polo <op@omarpolo.com>
- Date:
set_scroll_position: further semplifications
Simplify it even further. There's no need to keep the curfound variable
*at all*: if we end the loop, current_line is not found.
- Commit:
2ef8fcaedd93b1a56d0940ddd58a43a6d6c321f0
- From:
- Omar Polo <op@omarpolo.com>
- Date:
set_scroll_position: always set current_line
There's a small chance we may end up finding the topline but not the
current line if a page content is abruptedly changed.
While here also drop `curfound = 1': it's a dead assignment since it
returns in that conditions.
Bug found while reading the output of clang' scan-build, but not thanks
to clang. scan-build complained about the `curfound' variable, and by
re-reading the code I spotted the missing case.
- Commit:
638823f847d99e105ee6671fe1c51f8f048a0e48
- From:
- Omar Polo <op@omarpolo.com>
- Date:
don't reset scroll position if the cursor changed line
If the user managed to move the cursor before the page load, don't
restore the scroll position. This is to avoid unnecessary jumping and
causing possible confusion on the user.
- Commit:
e795e9355a607cbda3790e5c651c1a6f829e4326
- From:
- Omar Polo <op@omarpolo.com>
- Date:
save and restore scroll position
Save the offset to the first and current line of the buffers and restore
it when the page is loaded. This restores almost the exact viewport: we
save the offset of the *real* line in the document, but they get wrapped
before being displayed, so the actual restored position may be slightly
above from the original one (think of a long paragraph that gets wrapped
over multiple visual lines.) In practice, this isn't a drawback but
only an improvement over the current behaviour.
In the future we could save also the offset *inside* the line, to
restore the exact position, but that felt unnecessary for now.
Persisting the postion on the session file ensures that even when
re-opening telescope the scroll position is correct.
There are some very active pages, such as antenna or other aggregator,
where this feature doesn't really work well if you re-open the page
after a while, since new links are available, but otherwise is a god
sent!
See also github issue #1
- Commit:
77e9591fc27fd0f06aaff0cd94880ed1d5de9a75
- From:
- Omar Polo <op@omarpolo.com>
- Date:
fix a couple of printw fmt strings
two %d really needs %zu
- Commit:
28cfdf8ff8b69c0baf35b300d8c4c88f1c0d9007
- From:
- Omar Polo <op@omarpolo.com>
- Date:
add fringe prefix and allow to override offset for fringes alone
- Commit:
0aef305d12c879faacb155c7b7348e6dcd99f394
- From:
- Omar Polo <op@omarpolo.com>
- Date:
add vi-like `~' fringes after the end of the buffers
- Commit:
6fed00b04fadcd9bccf5671e72749512d6ac0d05
- From:
- Omar Polo <op@omarpolo.com>
- Date:
modeline: add an extra - after the trust status char
in part for symmetry, and in part because it seems a good spot to
add an indicator for the presence of a client certificate in the
future.
- Commit:
aa8228f3d90dee23e2f20242b326ca684a0244be
- From:
- Omar Polo <op@omarpolo.com>
- Date:
improve tabs rendering
- Commit:
bb28f1c2f6463f1e8881c90bb17a404cf7e5c3b7
- From:
- Omar Polo <op@omarpolo.com>
- Date:
move parse_last_session to the fs process
~/.telescope/session was parsed in the main process before dropping in
the sandbox: that's no good. This moves the initialization of the ui
events later in ui_main_loop and makes so that the fs process, after
entering the sandbox, parses and sends the content of the last session
back to the main process; when it's done the ui gets loaded and
telescope is back rocking.
I'm plannig to re-use the struct session_tab during the save session
operation soon.