Skip to content

Conversation

@eduardofcabrera
Copy link
Contributor

Proposed changes (including videos or screenshots)

The text content from chatbox goes to the file description when drag and drop a file.

Issue(s)

Steps to test or reproduce

2022-02-08-15-06-54.mp4

Further comments

@ostjen ostjen self-requested a review February 8, 2022 18:19
@ostjen
Copy link
Contributor

ostjen commented Feb 8, 2022

looks promising, just need to keep the text when the upload action is cancelled

ostjen
ostjen previously approved these changes Feb 9, 2022
Copy link
Contributor

@ostjen ostjen left a comment

Choose a reason for hiding this comment

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

tested here and LGTM, but I'll ask someone from the frontend team to review 😃

@dougfabris dougfabris changed the title [IMPROVE]: ChatBox Text to File Description [IMPROVE] ChatBox Text to File Description Feb 9, 2022
@eduardofcabrera eduardofcabrera merged commit 0192e7e into develop Feb 15, 2022
@eduardofcabrera eduardofcabrera deleted the file_description branch February 15, 2022 12:48
gabriellsh added a commit that referenced this pull request Feb 15, 2022
…age-template-2

* 'develop' of github.com:RocketChat/Rocket.Chat: (29 commits)
  Bump vm2 from 3.9.5 to 3.9.7 in /ee/server/services (#24509)
  [IMPROVE] ChatBox Text to File Description (#24451)
  Chore: Update Meteor to 2.5.6 (#24461)
  Chore: Update ws package (#24477)
  Bump underscore.string from 3.3.5 to 3.3.6 in /ee/server/services (#24498)
  Bump follow-redirects from 1.14.7 to 1.14.8 in /ee/server/services (#24491)
  Language update from LingoHub 🤖 (#24493)
  Covert to typescript the unarchive slash commands files (#24331)
  [IMPROVE] Add tooltips on action buttons of Canned Response message composer (#24483)
  Chore: Delete unused file (NewAdminInfoPage.js) (#24196)
  [IMPROVE][ENTERPRISE] Improve how micro services are loaded (#24388)
  [IMPROVE] Add return button in chats opened from the list of current chats (#24458)
  Bump express from 4.17.1 to 4.17.2 in /ee/server/services (#24469)
  Bump cookie from 0.4.1 to 0.4.2 in /ee/server/services (#24472)
  [IMPROVE] Close modal on esc and outside click (#24275)
  Chore(deps-dev): Bump ts-node in /ee/server/services (#24435)
  [IMPROVE] Add user to room on "Click to Join!" button press (#24041)
  [FIX] Implement client errors on ddp-streamer (#24310)
  Bump body-parser from 1.19.0 to 1.19.1 in /ee/server/services (#23963)
  Bump jaeger-client from 3.18.1 to 3.19.0 in /ee/server/services (#23961)
  ...
@pierre-lehnen-rc pierre-lehnen-rc mentioned this pull request Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants