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

Style top chunk/chat in Repartee differently (generalization of use context line in CPO) #1666

Open
jpolitz opened this issue Oct 20, 2022 · 3 comments

Comments

@jpolitz
Copy link
Member

jpolitz commented Oct 20, 2022

In CPO, the use context line is grayed out and requires extra interaction to edit.

@schanzer and I talked today and proposed that we have a convention that the first chunk in Repartee get similar treatment. It should be grayed out, not allow inserting new chunks above it, and require some extra step to edit it (probably through Options).

This could generalize the "don't edit above this line" message that's sometimes in starter files – if you don't want students messing with it, put it in the top chunk.

@asolove
Copy link

asolove commented Jan 1, 2023

How do you imagine this working so that the top section is easy to edit for teachers who want to add "do not edit" content, but not for students?

asolove added a commit to asolove/pyret-lang that referenced this issue Jan 1, 2023
@asolove
Copy link

asolove commented Jan 1, 2023

^ The mentioned commit handles the easy part of this for the top chunk:

  • styling it differently
  • making it not editable
  • removing the "+" above it

But it doesn't handle the actually-interesting part, where it becomes editable in some other way when you want to change the top chunk.

image

@schanzer
Copy link
Contributor

schanzer commented Jan 1, 2023

I imagine it might be as simple as "double-click to make editable", which assigns a class to that chunk that turns editing on. Once that chunk is saved, the class is removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants