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

Add 'Code Block' button to more markdown editors #40197

Merged
merged 3 commits into from
Apr 23, 2021

Conversation

Hamms
Copy link
Contributor

@Hamms Hamms commented Apr 21, 2021

Specifically, to the lesson Overview, Preparation, and Purpose field editors.

Also, the editor for the actitivity secion markdown field is starting to get a lot of buttons; we're going to want to make some changes there (and probably also do something to unify the editor we use for that with the one we use for these fields), so to start playing around with some ideas for what we could do I've collapsed the two-now-three buttons that we had in this editor down to a single dropdown! I think this will work much better, but either way doing this will let us get some early feedback from our editors.

Peek 2021-04-21 15-02

PR Checklist:

  • Tests provide adequate coverage
  • Privacy and Security impacts have been assessed
  • Code is well-commented
  • New features are translatable or updates will not break translations
  • Relevant documentation has been added or updated
  • User impact is well-understood and desirable
  • Pull Request is labeled appropriately
  • Follow-up work items (including potential tech debt) are tracked and linked

Specifically, to the lesson Overview, Preparation, and Purpose field editors.

Also, the editor for the actitivity secion markdown field is starting to get a _lot_ of buttons; we're going to want to make some changes there (and probably also do something to unify the editor we use for that with the one we use for these fields), so to start playing around with some ideas for what we could do I've collapsed the two-now-three buttons that we had in this editor down to a single dropdown! I think this will work much better, but either way doing this will let us get some early feedback from our editors.
@Hamms Hamms marked this pull request as ready for review April 21, 2021 22:04
@Hamms Hamms requested a review from a team April 21, 2021 22:04
<a
className="btn dropdown-toggle"
data-toggle="dropdown"
href="#"
Copy link
Member

Choose a reason for hiding this comment

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

FYI @madelynkasula that we are using bootstrap + jquery for dropdown functionality here and a few other places. this is probably an interesting case to consider re: component library. Not necessarily blocking for this PR, but you may be best positioned to share out whether it is important for us to avoid bootstrap functionality in react code. My understanding was that react was preferred over bootstrap and/or jquery, but I'm not sure we're very strict about that or whether there are negative implications in this case.

…out the rendering details of the MarkdownEnabledTextarea
@Hamms Hamms merged commit c0d5d4c into staging Apr 23, 2021
@Hamms Hamms deleted the add-programming-expression-button-to-more-editors branch April 23, 2021 16:31
@dmcavoy
Copy link
Contributor

dmcavoy commented Apr 23, 2021

Just a heads up that MarkdownEnabledTextarea is used in TextareaWithMarkdownPreview which shows on the Course and Script editor as well. It would be good to have follow up tasks to make sure the fields on Course and Script editor which now have this ability support it.

@Hamms
Copy link
Contributor Author

Hamms commented Apr 23, 2021

oh, good callout! The functionality in this change is pretty well-contained, but it does look like there are some different stylesheets on those pages which add some undesirable formatting
image

@dmcavoy
Copy link
Contributor

dmcavoy commented Apr 23, 2021

oh, good callout! The functionality in this change is pretty well-contained, but it does look like there are some different stylesheets on those pages which add some undesirable formatting
image

I also meant just that if I add a code block or resource in the script or course overview does it work correctly on the script and course overview pages?

@Hamms
Copy link
Contributor Author

Hamms commented Apr 23, 2021

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.

None yet

3 participants