Re: [AD] grabber indexing patch |
[ Thread Index |
Date Index
| More lists.liballeg.org/allegro-developers Archives
]
On Wed, Sep 18, 2002 at 09:37:23AM +0200, Eric Botcazou wrote:
> As I suggested in the original thread, I think an entry in the menu would be
> a better approach:
> "Indeed. It seems you went a little too far. What about adding instead an
> 'options' entry to the main menu, which would contain the 4 on/off options
> formerly on the top right of the screen ? Both the grid and compression
> settings would stay visible, as would any future option that is not an
> on/off setting (thus making it unsuitable for inclusion in the menu)."
>
> What I meant by "reworking" your patch is to turn your box into a menu
> entry.
Ok; I'm ok to do that myself to reduce the workload. If my version of the
grabber is still current, I can do it on friday, otherwise I'll wait for the
4.1.2 release.
I may also want to make a few changes to the datedit api (replace the
many parameters with a single struct, as Peter suggested, or a bitfield
for simple on/off flags, or both) that might make it easier to add new
functionality without breaking the grabber too badly.
If I still feel up to it by that time, I'll have another look at the
relative path option I suggested around the same time. I have it working
more or less ok, but it has some issues that I'll have to address before it
is ok.
Come to think of it, I also had something that added a pop-up box when
overwriting a symlink. Should I rework a version of that patch as well?
Hmm... looks like I have my weekend planned...
Evert