Keyboard commands don't check for editability before performing editing commands.

Bug #750762 reported by Connor Carney
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Scatterand
New
Low
Unassigned

Bug Description

We need to run all editing commands (enter in headings, indenting lists, navigating tables, etc.) through a single dispatcher that handles *all* of the keyboard shortcuts, that way we can check for editability just once before editing.

Connor Carney (cscarney)
Changed in scatterand:
importance: Undecided → Low
Connor Carney (cscarney)
Changed in scatterand:
milestone: none → 0.1
Revision history for this message
Connor Carney (cscarney) wrote :

The issue with the paragraphs is fixed, but the larger issue isn't fixed -- there is exactly the same problems with lists, and tables, and pretty much every editing feature we add in the future.

Right now, each editing feature registers its own keypress handler, then checks the context and performs the appropriate modifications. We need an infrastructure for scripts to register a combination of key, context, action, then have a single keypress handler where we can check the context *once* and dispatch it to the appropriate handler.

summary: - Pressing enter with the selection in a non-editable heading adds a non-
- editable paragraph
+ Keyboard commands don't check for editability before performing editing
+ commands.
description: updated
Connor Carney (cscarney)
Changed in scatterand:
milestone: 0.1 → 0.3
Connor Carney (cscarney)
Changed in scatterand:
milestone: 0.3 → 0.5
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.