Skip to content
This repository has been archived by the owner on Jan 6, 2023. It is now read-only.

"Take a note..." placeholder text is not displayed when erasing all text from the Notes content area #405

Closed
SoftVision-EmilPasca opened this issue Nov 20, 2017 · 8 comments
Labels
ckeditor help wanted [QA]:Wontfix issue Label for QA to mark issues that were decided by the team not to be fixed

Comments

@SoftVision-EmilPasca
Copy link

[Affected versions]:

  • Firefox 57 and up

[Affected Platforms]:

  • All Windows
  • All Mac
  • All Linux

[Prerequisites]:

  • Have a Firefox profile with the latest "Firefox Notes" add-on version (1.9.0b1 build with CKeditor) installed.

[Steps to reproduce]:

  1. Open the browser with the profile from prerequisites.
  2. Select all the text from the content area and delete it.
  3. Observe the "Take a note..." placeholder text.

[Expected result]:

  • "Take a note..." placeholder text is displayed.

[Actual result]:

  • "Take a note..." placeholder text is not displayed.

[Additional Notes]:

  • Attached a screen recording of the issue:
    placeholdertext
@Natim Natim added this to the Editor Sprint milestone Nov 20, 2017
@Natim
Copy link
Collaborator

Natim commented Nov 20, 2017

@vladikoff
Copy link
Contributor

Ref: ckeditor/ckeditor5#479

@Natim
Copy link
Collaborator

Natim commented Feb 2, 2018

@Natim
Copy link
Collaborator

Natim commented Feb 2, 2018

@anweshknayak Do you want to tackle this one?

@muddlebee
Copy link
Contributor

Hey @Natim sure 💯 . Thanks 😄

@Natim Natim removed this from the Editor Sprint milestone Feb 2, 2018
@muddlebee
Copy link
Contributor

muddlebee commented Feb 4, 2018

Sorry, I don't get this one. Take a note placeholder. What are you referring to?

@Natim
Copy link
Collaborator

Natim commented Feb 5, 2018

When the pad is empty (in the previous version notes) we had a placeholder telling people to take a note.

Since we moved to CKeditor we don't have it anymore.

I guess we need to plug this plugin: https://github.com/alexeckermann/ckeditor5-emptyness in order to add it back.

@vladikoff
Copy link
Contributor

[Expected result]:

"Take a note..." placeholder text is displayed.

We improved our "Welcome" copy and probably won't be bringing the old "Take a note..." placeholder unless CKEditor makes it really easy to add it.

@Softvision-MariusComan Softvision-MariusComan added the [QA]:Wontfix issue Label for QA to mark issues that were decided by the team not to be fixed label Jun 8, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
ckeditor help wanted [QA]:Wontfix issue Label for QA to mark issues that were decided by the team not to be fixed
Projects
None yet
Development

No branches or pull requests

5 participants