Skip to content
This repository was archived by the owner on Sep 6, 2021. It is now read-only.

sprint 30 draft#64

Merged
jasonsanjose merged 4 commits into
masterfrom
jasonsanjose/sprint30
Aug 26, 2013
Merged

sprint 30 draft#64
jasonsanjose merged 4 commits into
masterfrom
jasonsanjose/sprint30

Conversation

@jasonsanjose
Copy link
Copy Markdown
Member

@peterflynn
Copy link
Copy Markdown
Member

We have a couple things where it's not clear if they'll land in this sprint or not, e.g.:

  • Live HTML
  • "Default editor" file associations on first launch (Edge Code story)
  • Extensible linting API

Linting is just an API, so it's probably only worth mentioning in release notes.

"Default editor" would be worth mentioning if it lands, but I suspect it won't because it won't work well across Brackets sprint upgrades yet. So maybe it's not worth including even preemptively for translation?

"Open With" has the same problem with updates -- so I think we should remove it from this update notification. We shouldn't advertise it heavily until it's working more reliably...

Comment thread updates/stable/en.json Outdated
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.

Something like "Live Preview continuous HTML updating" might drive this home more -- you already can do 'HTML editing' in Live Preview in some sense...

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.

So wordy... 😬 but I can't think of anything better. "Live", "continuous" and "updating" all seem redundant to me.

@adrocknaphobia @njx @gruehle @dangoor Suggestions?

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.

Also, we'll keep this in for translation and remove if we must. I'll omit "open with" and "default editor" (not included in my last draft).

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.

How about "Live HTML Editing" or "Live Preview While Editing 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.

"Live HTML Editing" sounds pretty solid to me

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.

Done

@peterflynn
Copy link
Copy Markdown
Member

@jasonsanjose Just realized -- function parameter hints landed this sprint too. Seems worth including that. Is it too late?

(I just added this to the release notes draft if you want to borrow any of the phrasing there)

@jasonsanjose
Copy link
Copy Markdown
Member Author

@peterflynn sure that works

jasonsanjose added a commit that referenced this pull request Aug 26, 2013
@jasonsanjose jasonsanjose merged commit d562b76 into master Aug 26, 2013
@njx
Copy link
Copy Markdown

njx commented Aug 26, 2013

Might be too late, but the new Replace All functionality from @zaggino (I think) landed in this sprint--we should mention that as well. (It pops up a panel that lets you choose which items to replace.)

@jasonsanjose
Copy link
Copy Markdown
Member Author

Yep, that's from adobe/brackets#4686. I'll add it. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants