Skip to content

[FIX] Directory sort and column sizes were wrong#10403

Merged
rodrigok merged 7 commits intodevelopfrom
sort-discovery
Apr 19, 2018
Merged

[FIX] Directory sort and column sizes were wrong#10403
rodrigok merged 7 commits intodevelopfrom
sort-discovery

Conversation

@ggazzo
Copy link
Member

@ggazzo ggazzo commented Apr 10, 2018

we can`t sort by array length, (actually we can but its very expensive)

closes #9817
closes #10024
before:
image
after:
image

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-10403 April 10, 2018 14:37 Inactive
@ggazzo ggazzo requested a review from karlprieb April 10, 2018 22:06
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-10403 April 10, 2018 22:48 Inactive
@ggazzo ggazzo changed the title [FIX] better css table and disabled sort by users [FIX] scroll and sort on directory page Apr 10, 2018
@ggazzo ggazzo temporarily deployed to rocket-chat-pr-10403 April 10, 2018 22:50 Inactive
Copy link
Contributor

Choose a reason for hiding this comment

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

wrong indentation

@karlprieb karlprieb added the area: ui Touches the code on client side label Apr 11, 2018
@karlprieb karlprieb added this to the 0.64.0 milestone Apr 11, 2018
fix indentation
@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-10403 April 17, 2018 18:07 Inactive
karlprieb
karlprieb previously approved these changes Apr 17, 2018
@engelgabriel engelgabriel requested a deployment to rocket-chat-pr-10403 April 17, 2018 18:10 Abandoned
Copy link
Contributor

Choose a reason for hiding this comment

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

The Users is not i18n'd.

Copy link
Contributor

Choose a reason for hiding this comment

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

The Name is not i18n'd.

Copy link
Contributor

Choose a reason for hiding this comment

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

The Created At is not i18n'd.

@engelgabriel engelgabriel temporarily deployed to rocket-chat-pr-10403 April 18, 2018 20:31 Inactive
@karlprieb
Copy link
Contributor

@graywolf336 done :) Thank you for your review 👍

@rodrigok rodrigok changed the title [FIX] scroll and sort on directory page [FIX] Directory sort and column sizes were wrong Apr 19, 2018
@rodrigok rodrigok merged commit 981e150 into develop Apr 19, 2018
@rodrigok rodrigok deleted the sort-discovery branch April 19, 2018 12:30
MarcosSpessatto pushed a commit that referenced this pull request Apr 19, 2018
…-api-chat-postmessage-validations

* commit 'a9fb4da5c847a456990a5d60369f0f52ff4a8bd8': (137 commits)
  Remove "secret" from REST endpoint /settings.oauth response
  [FIX] Directory sort and column sizes were wrong (#10403)
  [FIX] Add oauth services missing fields, and indicate whether the oauth service is customized (#10299)
  Show error message when email verification fails (#10446)
  Correct the column positions in the directory search for users (#10454)
  Fixed custom fields misalignment in registration form (#10463)
  [FIX] Unique identifier file not really being unique (#10341)
  [OTHER] More Listeners for Apps & Utilize Promises inside Apps (#10335)
  [FIX] Empty panel after changing a user's username (#10404)
  [FIX] Russian translation of "False" (#10418)
  [FIX] Links being embedded inside of blockquotes (#10496)
  [FIX] The 'channel.messages' REST API Endpoint error (#10485)
  [OTHER] Develop sync (#10487)
  [FIX] Button on user info contextual bar scrolling with the content (#10358)
  [FIX] "Idle Time Limit" using milliseconds instead of seconds (#9824)
  [NEW] Body of the payload on an incoming webhook is included on the request object (#10259)
  [FIX] Missing i18n translation key for "Unread" (#10387)
  [FIX] Owner unable to delete channel or group from APIs (#9729)
  [NEW] REST endpoint to recover forgotten password (#10371)
  Add REST endpoint chat.reportMessage, to report a message (#10354)
  ...
@rodrigok rodrigok mentioned this pull request Apr 28, 2018
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Directory channel listing has incorrect behaviour in high Y resolutions Discovery sort ui doesnt update

5 participants

Comments