-
-
Notifications
You must be signed in to change notification settings - Fork 128
Chat toolbar #916
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Chat toolbar #916
Conversation
honestly i would only leave the burger there and the emoticons selector moved somewhere close to the mic icon like in popular messengers. the emots selector definitely shouldn't be hidden. and wrt to the characters counter @Neustradamus adores it.. I don't like it though. |
The toolbar has the emoticons icon and it's enabled by default. |
Well I definitely don't use it there coz I have the middle toolbar enabled. |
@stokito: Good catch for the problem if Chat or Groupchat toolbar is disabled. Several options exist, enable/disable/select are in Options -> Toolbars:
About "XMPP Account" info which is in Chat and Groupchat (MUC Room) tabs because this XMPP client is multi-accounts and it must seen directly to do not mix. Note: At the same time, there are several OMEMO problems with multi-accounts which must be solved since always. About "Character counter" (Message length) info which is in Chat tab, there is already an option to enable or disable: Options -> Messages -> Common -> Show character counter (at the same time, it is missing in Groupchat, must be added). |
Harmonization of "Actions" is needed, the icon must be same for Chat and Groupchat... |
@stokito: First, it is not because other XMPP Clients have not same UI that it is needed to do same. Harmonization of Chat and Groupchat "Actions" icon (menu or burger name) is needed but not the removal of the Chat or Groupchat toolbar etc. One advantage of Psi/Psi+ is that it can be customized (show/hide options), not removal features. Account label (lb_ident), Message length (lb_count) must not be removed... |
@stokito: About UI and OMEMO, OTR, OpenPGP icons, there are two tickets: |
@stokito: Please look screenshots of Chat and Groupchat toolbars:
OMEMO/OTR/OpenPGP icons are here. |
not quite, emoticons button is just something considered to be frequently used. Other stuff with disabled toolbars may go to the burger menu, but not the emots button. |
I noticed that the group chat has the burger menu with Actions.

If disable the toolbar above message editor with
options.ui.contactlist.toolbars.m1.visible
it also shows the Smiles picker which is IMHO better to remove from there - it's too far from the editor and honestly not really needed. Please tell me if you will be fine to remove it.But the personal chat doesn't have the Actions menu. It turned out that it's hidden if enabled the toolbar.

It should be always shown, similar menu is used in other clients so a user may expect it there. I removed the code to hide it and also made it look similar to the menu in the group chat.
Similarly it also shows the Smiles picker that is probably would be better to remove. The chars counter is probably also not used.