Skip to content

Comments

[FIX] Directory and Apps logs page#13938

Merged
sampaiodiego merged 7 commits intodevelopfrom
fix/directory
Apr 2, 2019
Merged

[FIX] Directory and Apps logs page#13938
sampaiodiego merged 7 commits intodevelopfrom
fix/directory

Conversation

@tassoevan
Copy link
Contributor

Closes #13909

@tassoevan tassoevan added type: bug area: ui Touches the code on client side labels Mar 28, 2019
@tassoevan tassoevan added this to the 1.0.0 milestone Mar 28, 2019
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13938 March 28, 2019 19:44 Inactive
@sampaiodiego
Copy link
Member

seems apps are using rc-directory class here .. it should be broken right?

@tassoevan
Copy link
Contributor Author

@sampaiodiego Awkward style reuse... Thanks for warning me.

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13938 March 28, 2019 22:39 Inactive
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13938 March 28, 2019 22:42 Inactive
@sudipt1999
Copy link

@tassoevan How css gets broken up here ! In my local environment I am could fix it by using 100% height to input field but it doesn't fix on @sampaiodiego .

class="rc-input__element js-search"
name="message-search"
id="message-search"
placeholder="{{_ "Search_Apps"}}"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Search_Apps is missing from i18n files

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13938 March 29, 2019 18:17 Inactive
sampaiodiego
sampaiodiego previously approved these changes Mar 29, 2019
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-13938 March 29, 2019 22:19 Inactive
@rodrigok
Copy link
Member

rodrigok commented Apr 1, 2019

@tassoevan can you fix the margins of the app's logs in this PR?

@rodrigok rodrigok temporarily deployed to rocket-chat-pr-13938 April 1, 2019 14:14 Inactive
@sampaiodiego sampaiodiego changed the title [FIX] Directory page [FIX] Directory and Apps logs page Apr 2, 2019
@sampaiodiego sampaiodiego merged commit 982ed73 into develop Apr 2, 2019
@sampaiodiego sampaiodiego deleted the fix/directory branch April 2, 2019 01:48
@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

area: ui Touches the code on client side type: bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants