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

[Bug]: Can't do a tabulation in the markdown editor (.md) #4389

Closed
4 of 8 tasks
azalty opened this issue Jun 25, 2023 · 3 comments
Closed
4 of 8 tasks

[Bug]: Can't do a tabulation in the markdown editor (.md) #4389

azalty opened this issue Jun 25, 2023 · 3 comments
Labels

Comments

@azalty
Copy link

azalty commented Jun 25, 2023

⚠️ This issue respects the following points: ⚠️

Bug description

Pressing tab makes the editor focus on the 3 dots icon instead of entering a tabulation when in the markdown editor (when editing a .md file)

This prevents me from making indented unordered lists like this:

  • Example
    • Example

Steps to reproduce

  1. Open a .md file
  2. Press tab

Expected behavior

Obvious enough, a tab should be entered, or it should at least be able to indent a list item

Installation method

None

Nextcloud Server version

27

Operating system

None

PHP engine version

None

Web server

None

Database engine version

None

Is this bug present after an update or on a fresh install?

None

Are you using the Nextcloud Server Encryption module?

None

What user-backends are you using?

  • Default user-backend (database)
  • LDAP/ Active Directory
  • SSO - SAML
  • Other

Configuration report

No response

List of activated Apps

No response

Nextcloud Signing status

No response

Nextcloud Logs

No response

Additional info

No response

@azalty azalty added 0. Needs triage bug Something isn't working labels Jun 25, 2023
@szaimen szaimen transferred this issue from nextcloud/server Jun 25, 2023
@mejo-
Copy link
Member

mejo- commented Jul 25, 2023

Dear @azalty, thanks for your report.

Entering tab characters is unsupported intentionally, to allow tab navigation on the website.

But about your second point: in lists, the Tab key should indeed indent the list item if appropriate. If this this doesn't work, it's clearly a bug. Unfortunately, I cannot reproduce this myself. For me, indenting list items with the Tab key works as intented.

In order to further track down the bug, we'd need some additional information:

  • Which browser and operating system do you use? Are you able to reproduce the bug with different browsers?
  • In case you have the Collectives app installed, does the bug also happen in Text documents there, or only if you open Text documents from the Files app?
  • If you share a single Text document via link and open this share in a private browser tab, are you able to indent lists with the Tab key there? Or does it still not work?

@github-project-automation github-project-automation bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Office team Jul 25, 2023
@Isac99
Copy link

Isac99 commented Jul 25, 2023

So only occurs when editing a .md file in the File Editor app.
Works fine in the Notes and Decks apps when editing rich text.
Using Microsoft Edge Version 115.0.1901.183 (Official build) (64-bit) on Windows 11. Also have witnessed the same behaviour in Google Chrome Version 114.0.5735.248 (Official Build) (64-bit)
Nextcloud 27.0.1

I opened the developer tools console on the browser and the following resource was not loaded caused by a CSRF check failed error message showing when I navigated to the resource:

ocs/v2.php/apps/text/workspace?path=%2FQueer+Stream%2FProduction%2FShows at at xhr.js:251

and for

index.php/apps/text/session/push at xhr.js:251 I receive a HTTP 405 error

and keep getting this message

index.php/apps/text/session/push 403 error xhr.js:251

Other messages
POST https://xxxxxx/index.php/apps/text/session/push 403
[ERROR] text: failed to write to document - not allowed {app: 'text', uid: 'xxx', level: 2}
[ERROR] text: Error: Failed to apply steps. Retry! {app: 'text', uid: 'xxx', level: 2}
Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'id')

Steps to reproduce:
Create a new .md file in a folder through the web interface

  1. Create an ordered or un-ordered list
  2. Write some text
  3. Press return to create a new line
  4. Press the tab button on the keyboard

Hope it helps... always happy to help out

Example URL
https://xxxxxxxxxxx.tv/index.php/apps/files/?dir=/Queer%20Stream/Production/Shows&openfile=15013

@juliusknorr
Copy link
Member

List item indentation is tracked in #3493 with a possible fix in #4735

@github-project-automation github-project-automation bot moved this from 🧭 Planning evaluation (don't pick) to ☑️ Done in 📝 Office team Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

4 participants