Skip to content
This repository was archived by the owner on Aug 31, 2021. It is now read-only.

[[ Bug 19890 ]] Remove the possibility of bringing up palettes for incorrect objects#1637

Merged
montegoulding merged 2 commits intodevelop-8.1from
feature-tutorial_force_selection
Jun 30, 2017
Merged

[[ Bug 19890 ]] Remove the possibility of bringing up palettes for incorrect objects#1637
montegoulding merged 2 commits intodevelop-8.1from
feature-tutorial_force_selection

Conversation

@livecodeali
Copy link
Copy Markdown
Member

  • Ensure object is always selected when step requires
    If an object waiting for a palette to open becomes deselected, go
    back to the selection step.
    Also, ensure this works for stacks by implementing similar code in
    ideActiveStacksChanged to ideSelectedObjectChanged

  • Ensure all palette for obj steps are preceded by select steps
    This is required for the enforcement of correct selection to work.
    Also, ensure all select steps highlight the object in question.

If an object waiting for a palette to open becomes deselected, go
back to the selection step.

Also, ensure this works for stacks by implementing similar code in
ideActiveStacksChanged to ideSelectedObjectChanged
…t steps

This is required for the enforcement of correct selection to work.
Also, ensure all select steps highlight the object in question.
@livecodeali livecodeali added this to the 8.1.6-rc-1 milestone Jun 23, 2017
@montegoulding
Copy link
Copy Markdown
Contributor

@livecode-vulcan review ok b1b9394

@livecodeali this looks fine but I can't help wondering if we should just have something like this that you can add to any step.

ensure <object> is selected [with message <info>]

@livecode-vulcan
Copy link
Copy Markdown

💙 review by @montegoulding ok b1b9394

@montegoulding montegoulding merged commit 51e2498 into develop-8.1 Jun 30, 2017
@livecodeali livecodeali deleted the feature-tutorial_force_selection branch June 30, 2017 08:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants