"Save and Share..." option in the file menu

[cycomachead]

This builds of @bromagosa's work. There is a now an item in the file menu
which when selected will save the current project to the cloud, then will
present the user with a text box to copy the URL.

This makes the following changes from what's currently in BettleBlocks:

• If the project hasn't been saved yet, then show a save dialog box first
• Presenting the URL is now done with a native morphic dialog box! Now that
  C&P works well, this looks much nicer. :)
• Adds a new IDE method publicProjectURL = function (username, projectName)
  • This gives the public project URL. I'm probably going to use this to
    refactor some existing code for getting URLs later.

There are some TODO elements:

• Definitely needs a thorough code review
• The dialog box looks a bit off. I think I could use some pointers here...
• Until we have context menus, we probably want some text to tell people
  how to copy text.
• Don't make a cloud request for the "share" service if the project is
  already public
• Update the browser's URL on successful sharing.

Deployed at http://snap.berkeley.edu/michael

[cycomachead]

Seenshot:

@brianharvey Would be good to confirm any wording concerns. :)

[brianharvey]

Nihil obstat.

[cycomachead]

Nihil obstat.

I had to look that one up! 👍

[cycomachead]

Fixed the layout with the dialog box! There's now a scrollbar so you can see the whole URL.

[bromagosa]

Oh I'm so stealing this one!

[cycomachead]

Jens, have you had a chance to look at this? I couldn't get some help text to easily show up, though I was probably missing something obvious...

[jmoenig]

nice! I'd like to play with it a little bit myself before pulling it into the main distro

[dekrain]

How ++ block works ?

[Klortho]

Status? I think this would be a huge improvement -- any chance of getting it pulled soon?

[dekrain]

I know now:

1. Change spec of {change %var by %n} to {change %repRing by %n}
2. Add block {%anyUE"val" ++}
3. Add definition: {
   {change (val) by (1)}
   {report (call (val))}
   }
4. =8D

[cycomachead]

@jmoenig @brianharvey

Any change of this being pulled?

[jmoenig]

Let's leave this one open for now...

[cycomachead]

As far as I know, it's mostly ready to go - is there interest in merging this?

[brianharvey]

It says here there's a conflict in gui.js so at least you have to fix that... But fwiw yes, I'd love to see this feature. It's a small thing but annoying not having it.

[cycomachead]

Yeah.

I can fix that. There's no point in fixing things that won't get merged though. :)

Michael Ball
From my iPhone
michaelball.co

On Oct 27, 2016, at 4:57 PM, Brian Harvey [email protected] wrote:

It says here there's a conflict in gui.js so at least you have to fix that... But fwiw yes, I'd love to see this feature. It's a small thing but annoying not having it.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

[cycomachead]

@jmoenig Any interest in putting this soon?

[Klortho]

👍