Commits
- Commit:
d08fccd2fe31fb4f2415704ca8e867dc4d2e7480
- From:
- Omar Polo <op@omarpolo.com>
- Date:
removed -C/--colours flag
- 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:
fd29e8735f8d2276b3dce90c267f67742bc5bd06
- From:
- Omar Polo <op@omarpolo.com>
- Date:
don't strlcat in a loop
not that of a big deal, but since we memset() at the start, just add
the spaces directly instead of using strlcat. saves some cpu cycles.
- Commit:
5ab1e3c1bda423908dde9e027f496f9d25f5e4da
- From:
- Omar Polo <op@omarpolo.com>
- Date:
fix wrapping of link lines
- Commit:
4592900367c6b286fb4de7ef1dbb70d991b753f9
- From:
- Omar Polo <op@omarpolo.com>
- Date:
fix comment
- Commit:
ad8e93f553f18c0c1f8e8963856b4a91c4438616
- From:
- Omar Polo <op@omarpolo.com>
- Date:
add missing void to please clang
- 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:
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.