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

Cog menu doesn't work for pages #1874

Closed
canstudios-nicolaw opened this issue Feb 14, 2018 · 0 comments
Closed

Cog menu doesn't work for pages #1874

canstudios-nicolaw opened this issue Feb 14, 2018 · 0 comments
Assignees
Labels
D: easy Straightforward issues, small time commitment S: merged Completed, reviewed, and merged issues T: bug

Comments

@canstudios-nicolaw
Copy link
Contributor

Expected Behaviour

The cog button on the page on the editor page view should open the context menu.

Actual Behaviour

Clicking the button does not open the context menu

Steps to Reproduce

  1. Create a course with a page
  2. Navigate to the edit page view
  3. Click the cog icon on the page

Versions

  • Authoring Tool Version: 0.4.1

Additional Information

All of the other cog icons work. Upon closer inspection it seems that the fix for this would be to remove the "openContextMenu" function from the editorPageView.js. The call to openContextMenu in this file will then use the "openContextMenu" function defined in editorOriginView.js.

@canstudios-nicolaw canstudios-nicolaw added T: bug D: easy Straightforward issues, small time commitment labels Feb 14, 2018
@canstudios-nicolaw canstudios-nicolaw self-assigned this Feb 16, 2018
@taylortom taylortom added the S: awaiting-review Completed issues waiting on reviews label Mar 5, 2018
@canstudios-nicolaw canstudios-nicolaw added S: merged Completed, reviewed, and merged issues and removed S: awaiting-review Completed issues waiting on reviews labels Mar 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D: easy Straightforward issues, small time commitment S: merged Completed, reviewed, and merged issues T: bug
Projects
None yet
Development

No branches or pull requests

2 participants