Pages:
Author

Topic: Bitcoin-Qt, the future Bitcoin client GUI [user input needed] - page 12. (Read 57177 times)

newbie
Activity: 24
Merit: 0
I'm still trying to get the Windows build working.  If anyone's able to answer this then it would be really helpful.

Current status: 1 build error which I think is due to the old version of BDB which I'm using (I had trouble compiling BDB under Windows so I found a pre-compiled version and used that.)  Leaving out the 2 header files which are causing a problem for me might have knock-on effects though so I don't think its as close to working as it looks.
Did you try to build using just mingw or cygwin instead of MS VS?
newbie
Activity: 48
Merit: 0
I'm still trying to get the Windows build working.  If anyone's able to answer this then it would be really helpful.

Current status: 1 build error which I think is due to the old version of BDB which I'm using (I had trouble compiling BDB under Windows so I found a pre-compiled version and used that.)  Leaving out the 2 header files which are causing a problem for me might have knock-on effects though so I don't think its as close to working as it looks.
newbie
Activity: 24
Merit: 0
Indeed, it should install an icon to the system directories anyway as application icon in the menus. Another reason to switch to an autoconf-based build system with a "make install", I guess.

For now we can just use the system default icons as notification.

NO! No autoconf! It is better to use cmake. Like all KDE or rpcminer is using. I had in future plan to take care about it so you can assign me this task. Additionally QtCreator also support it.
hero member
Activity: 812
Merit: 1022
No Maps for These Territories
The last screenshot is standard information from your program. I just rewrite all tray->showMessage to my Notification::instance->notify. For now I have biggest problems with this icons. Temporary I have hardcoded them, but in future they need to be get from system directories.
Indeed, it should install an icon to the system directories anyway as application icon in the menus. Another reason to switch to an autoconf-based build system with a "make install", I guess.

For now we can just use the system default icons as notification.
newbie
Activity: 24
Merit: 0
The last screenshot is standard information from your program. I just rewrite all tray->showMessage to my Notification::instance->notify. For now I have biggest problems with this icons. Temporary I have hardcoded them, but in future they need to be get from system directories.

About replacing all QMessageBox into notify, I also think that it isn't good idea. There are sometimes such situations when user must know that program want answer from him. But many messages can be done my path. For example information about changing status of transaction (from unknown to confirmed).
hero member
Activity: 812
Merit: 1022
No Maps for These Territories
zwierzak: cool. send a pull request once you think it's stable

I'm not sure that replacing *all* message boxes with notifications is a good idea, but for errors and notifications coming from the daemon/network thread it's a good thing.

However, for dialog boxes resulting from direct user interaction I'm not sure replacing it with a notification is how it's supposed to be (for example, in your last screenshot you use a notification to ask for fee confirmation)

Edit: Filter row has been committed now. Everything should work except for "Date -> Range...".
newbie
Activity: 24
Merit: 0
Hi,

I made some changes in BitCoin-Qt to support FreeDesktop Notifications (like KNotify in KDE4, libnotify in Gnome or osd-notify in Ubuntu). I think that they are a lot better then standard Qt messages and they're great integrating with system. If you don't read them now they will be stored for later (in KDE4). Only dependency is Qt-DBus, so we don't need in system kdelibs or libnotify. If your system don't support FreeDesktop Notify (you don't have daemon or you are using Windows) it will use at first Qt Tray messages and than QMessageBox.

Known issues:
- text is limited to 105 characters (probably limit in KNotify), so I need to made some possibility to read whole message,
- icons are hardcoded because DBus can't send icons' objects, it's require icon's path, which don't exists when we don't have installed program or we use Qt resources (they are inside binary file). I used Oxygen icons, but in future I will patch it to get icons from default system theme

Future:
- replace QMessageBox with FreeDesktop Notifications, you can add actions to notify, so they can replace QMessageBox in 100%, problem are asynchronous answers

Some screenshots:
http://img199.imageshack.us/img199/8586/bitcoinqt1.png
http://img850.imageshack.us/img850/2773/bitcoinqt2.png
http://img29.imageshack.us/img29/905/bitcoinqt3.png

If you're interested in my fork that go ahead and download it form my GitHub:
https://github.com/zwierzak/bitcoin-qt

When I finish my modification I'm going to push this changes to official repository of BitCoin-Qt, but not now. I must repair most annoying issues. Also I found some crashes in original program, which I will try to fix.

To encourage me to continue my work don't affraid to send some tips to my address:
1L7AeokogyuXGmUrx9SRTNhRPzYcmmghdo
XIU
member
Activity: 84
Merit: 10
Nope, was just talking about using absolute values, so that would be perfect Smiley
hero member
Activity: 812
Merit: 1022
No Maps for These Territories
That sounds cool. Maybe the window in step 3 could be the same window of the total list of addresses, with the new one selected.
Yeah, indeed, why not.
The solution for that is to remove "-l." from configure.ac
What is the equivalent with the QT SDK?
I'm not sure. It seems to be automatically added. Multiple times, even, looking at makefile  Roll Eyes

INCPATH       = -I/usr/share/qt4/mkspecs/linux-g++ -I. -I/usr/include/qt4/QtCore -I/usr/include/qt4/QtGui -I/usr/include/qt4 -Isrc -Isrc/json -Isrc/cryptopp -Isrc/qt -I. -I.

Nice work, would it be possible to add "BTC" to the balance? And will the amount column also work for the sent (negative) amounts?
I've added BTC to the balance. That'd indeed more clear.

Yes, the amount column uses the absolute amount and shows the transactions larger than that. If you want to select on received/sent, you can select with the 'Type' column.

It's not possible to search for transactions smaller than a certain amount. I'm not sure whether this is useful anyway, but it wouldn't fit into this interface to add a lower as well as an upper bound.
XIU
member
Activity: 84
Merit: 10
This is what I mean with a filter row, what'd ya think? Better than tabs isn't it?



This allows building basic filters:
  • Date column: allows selecting "All", "Today", "This week", "This month", "Range..."
  • Type column: allows selecting "Received from", "Sent to", "To yourself", "Generated", "Other"
  • Address column: type whole or partial address
  • Amount column: enter minimum size of transaction

I'm also going to add a "clear filter" broom button to the right.

Note that this is just a prototype still, please let me know your ideas and comments before I start implementing.


Nice work, would it be possible to add "BTC" to the balance? And will the amount column also work for the sent (negative) amounts?
newbie
Activity: 48
Merit: 0
I've hit a problem I haven't been able to solve yet in the Windows build.  Can anyone help?

I have the same problem building that Jaromil has here.

The solution for that is to remove "-l." from configure.ac
What is the equivalent with the QT SDK?
legendary
Activity: 1106
Merit: 1004
This is what I mean with a filter row, what'd ya think? Better than tabs isn't it?

Yes, definitively!

Thank you for your contributions to the project, it's getting better. Smiley
legendary
Activity: 1106
Merit: 1004
so, what workflow would you propose?

User wants to receive coins from someone.

1. He clicks "New address"
2. He enters label for address and presses enter/confirms
3. A window appears from which the new address that can be copied to the clipboard/Drap and dropped.
   He is reminded that the address can be found in the 'Received addresses' list later.

Something like that?

That sounds cool. Maybe the window in step 3 could be the same window of the total list of addresses, with the new one selected.
hero member
Activity: 812
Merit: 1022
No Maps for These Territories
This is what I mean with a filter row, what'd ya think? Better than tabs isn't it?



This allows building basic filters:
  • Date column: allows selecting "All", "Today", "This week", "This month", "Range..."
  • Type column: allows selecting "Received from", "Sent to", "To yourself", "Generated", "Other"
  • Address column: type whole or partial address
  • Amount column: enter minimum size of transaction

I'm also going to add a "clear filter" broom button to the right.

Note that this is just a prototype still, please let me know your ideas and comments before I start implementing.
hero member
Activity: 812
Merit: 1022
No Maps for These Territories
The recommended way to use bitcoin is by creating a new address for each receipt.
Hmm yeah you're right, the current GUI doesn't really help in the recommended workflow (and that's what a GUI should do). Maybe add "New receiving address", "Your receiving addresses" to the toolbar and drop the address from the main screen then.
Quote
Wouldn't a combo become too big eventually? Anyway, if it is easy to filter by the address label I suppose that's not a problem.
Good point... edit: so, what workflow would you propose?

User wants to receive coins from someone.

1. He clicks "New address"
2. He enters label for address and presses enter/confirms
3. A window appears from which the new address that can be copied to the clipboard/Drap and dropped.
   He is reminded that the address can be found in the 'Received addresses' list later.

Something like that?
legendary
Activity: 1106
Merit: 1004
Hm you have a point. On the other hand I think it's very useful to have an address on the main page, and you can quickly copy it and give it to someone.

The recommended way to use bitcoin is by creating a new address for each receipt. Somebody following this practice will already have to click a button to generate a new address in a new window. In this window it's already possible to copy it to the clipboard.
I don't think the behavior of not following this practice should be facilitated by the client, as it is today, by making the last address there, available to be copied and reused.

Or change it to a combo box if there are multiple receiving addresses, so you can directly select which default address you want? That's probably most clear...

Wouldn't a combo become too big eventually? Anyway, if it is easy to filter by the address label I suppose that's not a problem.
hero member
Activity: 812
Merit: 1022
No Maps for These Territories
I think that would be good because frequently I've seen people who think that address which is being shown there is the bitcoin addresses of theirs. Like if there were only one, as a bank account number. This happens frequently.
Hm you have a point. On the other hand I think it's very useful to have an address on the main page, and you can quickly copy it and give it to someone.

I propose an alternative: instead of having a "New..." button next to the address, make it an "Add..." button (with tooltip "Add new receiving address"). This makes it clear that the old address is not being replaced.

And maybe change "Your Bitcoin address" to "Current Bitcoin address"?

Or change it to a combo box if there are multiple receiving addresses, so you can directly select which default address you want? That's probably most clear...

legendary
Activity: 1106
Merit: 1004
Nice to see such improvements! Great work!

I'll suggest a change that I think would make some newbies understand bitcoin addresses faster: remove the last selected address from the main window. Make the user always have to click somewhere to open the list of addresses, even if it only has one.

I think that would be good because frequently I've seen people who think that address which is being shown there is the bitcoin addresses of theirs. Like if there were only one, as a bank account number. This happens frequently.

By not having this one address on the main page, they will have to search it somewhere else, and there they will see that there is space for an entire list, with buttons to "create new" and all that.
hero member
Activity: 812
Merit: 1022
No Maps for These Territories
I was using 0.3.22 prior. made a backup of my wallet. tried this. went back to 0.3.22 and my wallet is corrupt. put my old wallet back and still wont work. now im forced to use this?
This was in the README :/
Quote
Berkely DB version warning

A warning for people using the static binary version of Bitcoin (tl;dr: Berkely DB databases are not forward compatible).

The static binary version of Bitcoin is linked against libdb4.7 or libdb4.8 (see also this Debian issue).

Now the nasty thing is that databases from 5.X are not compatible with 4.X.

If the globally installed development package of Berkely DB installed on your system is 5.X, any source you build yourself will be linked against that. The first time you run with a 5.X version the database will be upgraded, and 4.X cannot open the new format. This means that you cannot go back to the old statically linked version without significant hassle!
You can go back but you'll have to build it from source and link against the same berkelydb...
newbie
Activity: 48
Merit: 0
For anyone else trying to do a Windows build:
Compiling libboost isn't as easy as it could be.  After downloading the sources, edit bootstrap.bat and change the lines
Quote
call .\build.bat > ..\..\..\bjam.log
and
Quote
set toolset=msvc

to read

Quote
call .\build.bat gcc > ..\..\..\bjam.log
and
Quote
set toolset=gcc
Pages:
Jump to: