updating nickname should not require pressing enter

Bug #467024 reported by Toni Ruottu
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenFilepad
Fix Committed
Critical
Unassigned

Bug Description

Currently when a user updates her nickname she is required to press enter. We should probably have some buttons for updating the nickname, but doing correctly requires some thought.

Changed in qfriendslol:
status: New → Confirmed
milestone: none → 0.2.1
importance: Undecided → Wishlist
Revision history for this message
Toni Ruottu (toni-ruottu) wrote :

We just had another user get tricked by this one. Luckily he was still online when I noticed it.

Revision history for this message
Toni Ruottu (toni-ruottu) wrote :

And again! I changed importance to critical. It seems this bug renders our software mostly unusable. This time I had even forgot about the bug and it took me more than an hour to figure what went wrong.

Changed in qfriendslol:
importance: Wishlist → Critical
milestone: 0.3.1 → 0.2.1
status: Confirmed → Triaged
Revision history for this message
Sami Saada (samitheberber) wrote :

This is my suggestion for this bug. It works so that the written text is send, when Qt thinks that the field is changed. Both user edited and program edited ways work. Give some thoughts about it :)

Revision history for this message
Toni Ruottu (toni-ruottu) wrote :

Technically it works. Writing "toni" as my nickname causes the application to advertise me after each letter. So it advertises me as "t", "to", "ton", and "toni". And possibly some other names, if I make mistakes while typing. This wouldn't be that much of a problem, but sometimes OpenLookup may be under lag, if we pick one of the lighter servers. Advertising the nickname after each change will have a negative impact on user experience, as the user will experience the lag always after typing a letter. :-/

Revision history for this message
Sami Saada (samitheberber) wrote :

Second try, lets make button, which activates this functionality.

Revision history for this message
Toni Ruottu (toni-ruottu) wrote :

Seems to work. I applied the patch to our main branch.

Changed in qfriendslol:
status: Triaged → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.