Re: [Flamerobin-devel] BLOB-Editor as Dialog - Patch 001

[Milan B. <mil...@gm...>]

On Tue, May 26, 2009 at 9:50 PM, Andreas Maier <an...@we...> wrote:
>>> Personally i would prefer to "chech" data on loading. If a 0 char is
>>> detected text-tab will be disabled.
> Ups a typo - i meant "check".

Ah, that makes sense now.

In my experience, people generally do follow the rules and use
sub_type zero for binary data and sub_type one for text. Sometimes
they use something else (like -1 or 10 or whatever) to mark special
kind of binary data. IMHO, we should follow the simple rule and do
what is normally expected. If some users use Firebird in some way that
is not "standard", it's their problem. Therefore:

- if subtype is 1: text is the default tab and editing in "text" tab is enabled
- if subtype is 0: binary is the default tab and editing in "text" tab
is disabled

Later we can extend this to try to autodetect HTML and Images (JPG,
BMP, PNG, etc.) once those tabs are added.

> I don't know ATM how to handle. Yes copy/pasting would be good. But the
> text will be truncated if you save it back.

I meant "copy" in FlameRobin and "paste" in some other application.
When control is read-only you cannot paste into it anyway.

> Maybe i could handle it as follows:
> 1) Detect characterset of blob / database

No need for this. All data FR (or any other client for that matter)
receives has "connection charset" encoding.

> BTW: "cache" mechanism is already implemented. If you edit text and
> switch to binary the binary field will show the edited text.

I see. Ok.

BTW, one more thing I'd like to see is the ability to resize and
maximize the dialog.

-- 
Milan Babuskov
http://www.flamerobin.org