Fix id dropdowns - #4547
Merged
Merged
Conversation
davidsbailey
force-pushed
the
fix-id-dropdowns
branch
from
October 14, 2015 01:40
62bc020 to
04ac1cd
Compare
Member
Author
There was a problem hiding this comment.
the problem with pulling these IDs from design mode was that they were wrapped in the draggable/resizable wrapper. It seems easier to just pull these from divApplab which will have been recently updated from designModeViz. As a bonus, students who click the dropdown after the program has run will additionally see dynamically created element ids.
Contributor
|
ugh, changed my filtering rules for git stuff, and apparently being assigned PRs doesnt end up in my inbox now. if you assign me something and i don't respond, feel free to poke me. changes lgtm |
deploy-code-org
added a commit
that referenced
this pull request
Oct 14, 2015
commit d1b8e20 Merge: 30a0c41 6be1903 Author: Trevor Berg <trevor@code.org> Date: Tue Oct 13 21:25:20 2015 -0700 Merge pull request #4533 from code-dot-org/autocomplete-tooltips-rebase Add the ability to not show droplet tooltips on a per level basis commit 30a0c41 Merge: 60ed967 04ac1cd Author: davidsbailey <davidsbailey@users.noreply.github.com> Date: Tue Oct 13 21:24:41 2015 -0700 Merge pull request #4547 from code-dot-org/fix-id-dropdowns Fix id dropdowns commit 60ed967 Author: Continuous Integration <dev@code.org> Date: Wed Oct 14 04:19:55 2015 +0000 Automatically built. commit 91bd150 Merge: 3acdc4f 1db88fa Author: Trevor Berg <trevor@code.org> Date: Tue Oct 13 21:10:06 2015 -0700 Merge pull request #4531 from code-dot-org/fix-debug-icon Get rid of hacks for debugger hide/show state commit 3acdc4f Merge: 867ec7f 1c7c553 Author: Trevor Berg <trevor@code.org> Date: Tue Oct 13 21:08:46 2015 -0700 Merge pull request #4548 from code-dot-org/xframe Rewrite documentation urls to forward on the embedded flag commit 867ec7f Author: Continuous Integration <dev@code.org> Date: Wed Oct 14 01:05:32 2015 +0000 Automatically built. commit 9764e23 Merge: b37971a 4842714 Author: Elijah Hamovitz <elijahhamovitz@gmail.com> Date: Tue Oct 13 18:00:05 2015 -0700 Merge pull request #4459 from code-dot-org/readonly-function-editor Add Blockly.readOnly check to function editor commit b37971a Author: Josh Lory <josh.lory@code.org> Date: Tue Oct 13 17:46:30 2015 -0700 Cucumber doesn't like inline comments commit 91bd150 Merge: 3acdc4f 1db88fa Author: Trevor Berg <trevor@code.org> Date: Tue Oct 13 21:10:06 2015 -0700 Merge pull request #4531 from code-dot-org/fix-debug-icon Get rid of hacks for debugger hide/show state commit 3acdc4f Merge: 867ec7f 1c7c553 Author: Trevor Berg <trevor@code.org> Date: Tue Oct 13 21:08:46 2015 -0700 Merge pull request #4548 from code-dot-org/xframe Rewrite documentation urls to forward on the embedded flag
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes regressions in id dropdowns outlined in https://www.pivotaltracker.com/n/projects/1256654/stories/105621516, introduced by #4464.
This needs some tests, but I also want to make sure it gets merged in time for morning bug bash.