Skip to content
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

[FIX] Remove Room info for Direct Messages (#9383) #12429

Merged
merged 9 commits into from
Mar 15, 2019

Conversation

vinade
Copy link
Contributor

@vinade vinade commented Oct 26, 2018

To solve this issue I had to do 3 things:

  • Remove the Room Info button from TabBar when room type is Direct Messages.
  • Create another E2E toggle button outside Room Info
  • Disable E2E toggle button for public rooms

The main point of this issue was simple to solve, but it created another issue.
The E2E toggle button is inside Room Info, and if there is no Room Info then the user cannot reach the E2E button.

So, I'm proposing a second E2E toggle button beside Favorite toggle button at header. Like this:
E2E off : captura de tela de 2018-10-26 01-44-06

E2E on :
captura de tela de 2018-10-26 01-43-20

The new toggle button replaces the old E2E icon (which was only showed if E2E was on).
Now the button is always shown. And the button color shows the E2E state (green for on and grey for off), like the Favorite toggle button.

The last item is another issue I found testing. Later I searched on Github and I found that someone discovered it before ( #12417 ).
I noticed that E2E cannot be enabled at Public rooms. It always shows an error message.
And it seems to be done on purpose (to not be possible to enable E2E for public rooms). So I removed this option for Public rooms from interface.

Public room wihout E2E button
captura de tela de 2018-10-26 01-44-17

I had to solve this last issue because with the new E2E button this issue would become more frequent.

Closes #9383

@ggazzo ggazzo force-pushed the room-info-challenge branch from 82b3b94 to 47528a7 Compare February 7, 2019 19:11
@ggazzo ggazzo added this to the 1.0.0 milestone Mar 1, 2019
@ggazzo ggazzo merged commit e4bb492 into RocketChat:develop Mar 15, 2019
@rodrigok rodrigok mentioned this pull request Apr 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[PR8411] [BUG] [UI] [Discussion] We don't need Room Info for direct messages
3 participants