Scrollbar in chat window is wrong (#128)
Closed
wojciech.kapcia@tigase.net opened 5 years ago

When testing #issue #125 i noticed oddity - scrollbar is rather useless currently:

  • some chats don't have it at all (which makes it harder to orient oneself in window)
  • some chats have it but it's size and position doesn't correspond at all with what's displayed.

AFAIR you stated, that due to dynamic loading messages from server scrollbar is a "no go" but in that case it should be removed completely.

However, I'd argue that it's useful (even with dynamically loading messages from server and with that adjusting length of chat) to orient oneself about position and (relative) amount of messages/dialogue length.

beagle-scrolbar-small.mov beagle-scrollbar.png

Andrzej Wójcik (Tigase) commented 5 years ago

OK, let's start with a few notes.

  1. Scrollbars are disabled in XCode and I was not able to get them in the chat view. It looks like there is an issue with rendering UI on your side.
  2. After I've enabled them, they behaved correctly - proper size and position.

For the next build, I've decided to enable them and you should check them as there were some concurrency issues in the app, which are now solved and manipulation of the chat view from different threads could lead to the reported issues.

However, I'm not convinced that we should have scrollers enabled.

wojciech.kapcia@tigase.net commented 5 years ago

Currently (in b49) I have two scrollbars: one correct (I guess the new one) and one wrong (I guess the one I had before).

wojciech.kapcia@tigase.net commented 5 years ago

comment from chat: it's the member list scrollbar (which is not hidden despite actual member list not being shown)

Andrzej Wójcik (Tigase) commented 5 years ago

The issue is now fixed and will be part of the next build b50

issue 1 of 1
Type
Bug
Priority
Normal
Assignee
Spent time
2h 45m
Issue Votes (0)
Watchers (0)
Reference
tigase/_clients/beagle-im#128
Please wait...
Page is in error, reload to recover