97eae5c9b366002d3d9cc73e4955176b58110347
[elpher.git] / ISSUES.org
1 #+TITLE: Development notes/ideas
2 #+TODO: OPEN(o!) | CLOSED(c!) INVALID(i@)
3 #+STARTUP: logdrawer
4    
5 * Open Bugs
6
7 * Closed Bugs
8   
9 ** CLOSED Org mode faces are not present in recent emacs versions
10 Even 26.1 doesn't seem to have these.  This means that, for many
11 users, elpher doesn't show any difference between any of the
12 item types.  Not a major problem at all, but the faces we inherit
13 from should definitely be ones which have been present for much
14 longer.  Perhaps the font lock mode faces are the way to go after
15 all.
16
17 Update: changed all default faces to inherit from font-lock and basic faces.
18
19 ** CLOSED URL-centric addressing breaks bookmark file compatibility
20    
21 Need a way to allow people to rescue their old bookmark files
22 following this update.
23
24 ** CLOSED History loops <2019-11-08 Fri>
25
26 Occasionally elpher gets stuck in a "history loop" where a
27 node is its own grandparent.  Obviously this sucks, as history
28 is elpher's main mechanism for making gopherspace exploration
29 painless.
30
31 I suspect the problem is in either ~elpher-visit-node~ or
32 ~elpher-visit-parent~.
33
34 Follow-up: this has been fixed by the new stack-based history system
35 in 2.5.
36
37
38 ** CLOSED Redirects do not rewrite current address
39
40 This is a bug, as gemini://blah.com/hi may get redirected
41 to gemini://blah.com/hi/, at which point link lines
42 of the form "=> there" should be interpreted as pointing
43 at gemini://blah.com/hi/there, while currently they are
44 interpreted as pointing at gemini://blah.com/there.
45
46 ** CLOSED History inconsistency when restarting elpher <2020-05-26 Tue>
47
48 To reproduce:
49 1. open elpher and follow a few links until you're a handful of links below
50    the start page.
51 2. kill the elpher buffer with C-x k
52 3. Open elpher again, which will show the start page.
53 4. Press 'u' to go up.  Elpher wiill respond stating that there is no previous page.
54 5. Press 'u' again. Elpher will then jump to the page that was open when
55    the buffer was originally killed.
56
57 Expected behaviour: elpher should be once again at the bottom of the history
58 stack and should not remember the previous history.
59
60 Observed behaviour: elpher _does_ remember the previous history.
61
62 *** update <2020-05-27 Wed>
63 Turns out this was just because the `elpher` function was merely setting
64 the `elpher-current-page` variable to nil, then using `elpher-visit-page`
65 to visit the start page, resulting in the nil being pushed onto the existing
66 history stack.  Because `elpher-visit-previous-page` always trys to pop from
67 this stack and tests whether the result is nil (which it is when the stack is empty),
68 the first "u" would result in the "no previous page" message but would still
69 pop the stack, meaning that subsequent "u" commands would succeed.
70
71 The fix is just to zero out the history list in the `elpher` function just as
72 `elpher-current-page` is cleared.
73
74 * Open Enhancements
75
76 ** OPEN Allow multiple elpher buffers [33%]
77
78    Shouldn't be too hard, just need elpher-current-node to be
79 buffer-local and allow various buffer-switching procedures to
80 do something sensible.
81
82 Here are the things that need to be implemented before
83 this can happen:
84 - [X] shift history out of node tree and into separate stack
85 - [ ] make history stack variables buffer-local
86 - [ ] have elpher-with-clean-buffer select appropriate buffer 
87
88 ** OPEN Replace support for user-specified starting pages
89 This used to be available, but was removed during a refactor.
90
91 ** OPEN Allow for grouping of bookmarks
92 To support this I'd like to add a bookmark page specific
93 set of keybindings.  Currently all bindings available on
94 the bookmark page are available everywhere else.  But
95 expanding and collapsing bookmark groups sounds like it
96 might need more specific bindings.
97
98 *** Priority bump <2020-05-31 Sun>
99
100 As bookmark lists grow, some sort of grouping is becoming more and more
101 important.  Furthermore, with this in place it would become feasible
102 (and I really suspect almost trivial) to implement an update-checking
103 system for chosen groups of bookmarks.
104
105 For instance, we could prefetch content for each of the addresses within
106 a chosen group, indicating which had been changed since the last fetch.
107 (We could just store hashes of earlier content to detect changes.)
108
109 The difficult thing to decide is how the UI for the new bookmark page
110 will work.  It already has its own renderer, and we could easily stop
111 using the gopher directory line renderer in favour of something more
112 amenable to displaying the group information.  Thus we're very free to
113 do whatever we like once we also have a special key map in place as well.
114
115 I guess I need to look into what native widgets Emacs has for displaying
116 collapsable hierarchies.
117
118 ** OPEN Add history browsing
119
120 * Closed Enhancements
121   
122 ** CLOSED Turn on lexical scoping
123
124    A branch exists for this, but there are some compilation kinks
125 to iron out.
126
127   
128 ** CLOSED Implement support for telnet entries
129
130 Similar to http entries, telnet entries will be handled by code
131 external to elpher. However it seems I made http entry handling a
132 special case, and I don't want another!  So the only option is to
133 bring both http and telnet entries back into the fold by representing
134 them both as standard nodes and having the grunt work done by getter
135 functions.
136
137 ** CLOSED Allow users to access selected and current node details.
138    
139 ** CLOSED Implement bookmark system
140
141   Currently the bookmark page replaces the current page, and it
142   does so silently (i.e. it doesn't become part of the link hierarchy).
143   I think this is a mistake, as it results in confusing behaviour when
144   traversing the link hierarchy after visiting one of the bookmarked links.
145
146   Instead, I think I should
147   1. Make the bookmark page part of the hierarchy, and
148   2. Reinstate the visited node hash table to avoid excess link hierarchy pollution.
149
150   In order to accomplish 1. it will be necessary to make the bookmark page renderer
151   a proper getter function, and one that never caches the contents of the buffer.
152
153   Actually, I might have to think about that a bit more.  I don't know
154   how to answer the question of what the best thing to do with node
155   parent links when using a cached node in place of a new node.  (Maybe
156   I always update node.parent unless parent is already an ancestor of
157   node?)
158
159   
160 ** CLOSED Support character encoding diversity
161
162 ** CLOSED Make URLs the basic address type.
163 Currently I waste a lot of effort converting between
164 URL and non-URL representations.  This is unnecessary, and
165 actually makes lots of things uglier.
166
167 For example, the bookmarks file contains addresses in Elpher's
168 internal representation, whereas I expect users would prefer
169 it contain URLs.
170
171 So the idea would be for (elpher-node-address node) to be
172 a either a string or a symbol, with symbols used for "special"
173 pages (bookmarks, start page, etc).  The getter functions
174 `elpher-address-selector' etc will still do what they currently
175 do, but will process the URL to do it.
176
177 This also means that non-gopher URLs will be explicitly represented
178 as such: no more abusing the "h" type for these.
179
180 ** INVALID Remove "redraw" command
181 This is only necessary for returning from displaying the raw
182 server response.  If I can provide a better way of doing that
183 then we can get rid of redraw entirely.
184
185 Actually, this command can be useful to correct rendering issues that
186 occasionally pop up in termal windows.  Lets leave it for now.
187
188 ** CLOSED Implement Finger support
189    
190 ** CLOSED Improve download performance
191    This is actually easy to fix - the major problem at the moment is
192    the braindead way the incrementally-retrieved data is recorded:
193    (setq result-string (concat result-string next-bit)).
194    This is O(N^2).  Yuck!
195    
196    Okay, replacing this really does improve things.  Large gemini
197    downloads now seem occur at rates I'd expect.
198    
199 ** CLOSED Download/rendering progress feedback
200    Particularly for large files or complicated pages, elpher can
201    take a few seconds or more to generate a response.  Thhis is
202    frustrating for users, who are left staring at a blinking
203    cursor.
204
205    A small amount of feedback could help with this.
206
207 ** CLOSED Implement Gemini support [100%]
208    :LOGBOOK:
209    - State "CLOSED"     from "OPEN"       [2020-06-20 Sat 22:32]
210    :END:
211    
212 Here is the checklist of features required before release:
213 - [X] basic genimi transactions
214 - [X] gemini transactions requiring client certificates
215 - [X] gemini input handling
216 - [X] gemini map files (text/gemini)
217 - [X] Support for plain text responses (text/*)
218 - [X] Support for image responses (text/image)
219 - [X] Support for mime-specified character encodeing
220 - [X] Saving responses to disk
221 - [X] Viewing raw responses
222   
223 The last few will be made infinitely easier if we factor the
224 gopher "getter" code differently.