Skip to content

Content libraries UI updates - #14

Closed
mtyaka wants to merge 4 commits into
content-libraries-prototypefrom
content-libraries-ui-updates
Closed

Content libraries UI updates#14
mtyaka wants to merge 4 commits into
content-libraries-prototypefrom
content-libraries-ui-updates

Conversation

@mtyaka

@mtyaka mtyaka commented Nov 17, 2014

Copy link
Copy Markdown
Member
  • On the dashboard page, the blue border now appears only when the state is active. On hover for an inactive link the text color gets changed to black, but the bar is not shown.
  • Update the copy of the text on the new library form to match the design and removed the beta warning on top of the form.
  • Remove ability to edit the name of the library container on the top left of a library page and make display_name library field non-editable.

This changes the style of the inactive libraries/courses tab when hovering.

Only the active tab on the dashboard page gets the blue underline.
When hovering over an inactive tab, the color of the text changes,
but the underline does not appear.
@mtyaka mtyaka changed the title Content libraries ui updates Content libraries UI updates Nov 17, 2014
@mtyaka
mtyaka force-pushed the content-libraries-ui-updates branch from 7a0f468 to 8eb8f7b Compare November 20, 2014 07:40
Comment thread cms/templates/index.html

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mtyaka "This cannot be changed" => not true anymore, since this can still be changed from the Edit menu right?

Btw I've confirmed that we want to keep it editable in that menu for now. Btw I don't see that edit menu on the sandbox instance that runs https://github.com/edx/edx-platform/pull/6046 -- http://content-libraries.sandbox.opencraft.com:18010 Did I miss something or is this menu dependent on another PR?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@antoviaque Nice catch, I'll remove "This cannot be changed". I'm have no idea why the button doesn't show up on that instance.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mtyaka Sounds good, thanks!

@e-kolpakov Cf above - do you know why the "Edit" doesn't show on the sandbox based on https://github.com/edx/edx-platform/pull/6046 ?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@antoviaque because @bradenmacdonald removed edit button from content_libraries/2-studio-lib-support for some reason. I'm not sure though if it was planned to be added later, or a mistake.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@e-kolpakov Ok, thanks for the info. Unless they block about this in the upstream PR, let's wait until @bradenmacdonald comments on this when he's back then.

@mtyaka

mtyaka commented Dec 4, 2014

Copy link
Copy Markdown
Member Author

Some of the changes from this PR were already cherry-picked into content_libraries/2-studio-lib-support by @bradenmacdonald. I opened a new PR for the changes that didn't make it in yet at #17 and am closing this one.

@mtyaka mtyaka closed this Dec 4, 2014
@mtyaka
mtyaka deleted the content-libraries-ui-updates branch January 8, 2016 08:01
GustavoKatel pushed a commit that referenced this pull request Nov 22, 2018
Update requirements to pull in edx-olive-oc version of ORA2.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants