vscode-ocaml-platform icon indicating copy to clipboard operation
vscode-ocaml-platform copied to clipboard

Change logo and name

Open tmattio opened this issue 3 years ago • 12 comments

This PR addresses part of the concerns raised in #457:

  • Change the logo to the official OCaml logo
  • Change the name of the extension to OCaml

For the second point, the name currently collides with @hackwaly's extension, but he agreed to deprecate it in favor of this one, so if https://github.com/hackwaly/vscode-ocaml/pull/124 is merged with the name update, that won't be a problem anymore.

Merge blocked by the PR above, and a new release of hackwaly/vscode-ocaml.

tmattio avatar Nov 30 '20 13:11 tmattio

Is changing the extension ID from ocamllabs.ocaml-platform something that we want to do right now? I think it would be listed as a different extension on the marketplace and make it harder for existing users to update.

If this repo ever moves to the https://github.com/ocaml organization, maybe we could change the ID to ocaml.ocaml then?

mnxn avatar Nov 30 '20 20:11 mnxn

I think it would be listed as a different extension on the marketplace

I don't think that's the case. The PR I mentioned renamed the extension ID from ocaml to ocaml-deprecated and the old extension is not listed in the market place anymore: https://marketplace.visualstudio.com/search?term=ocaml&target=VSCode&category=All%20categories&sortBy=Relevance

tmattio avatar Nov 30 '20 21:11 tmattio

That extension is still listed under hackwaly.ocaml: https://marketplace.visualstudio.com/items?itemName=hackwaly.ocaml

hackwaly.ocaml-deprecated is a 404: https://marketplace.visualstudio.com/items?itemName=hackwaly.ocaml-deprecated

I don't see how it would be possible for the vsce tool to detect an extension ID change since it reads from the package.json to decide what extension it's publishing in the first place.

If we're planning to move to the ocaml organization/publisher (are we?), I'd prefer to wait until then to change the extension ID in order to minimize changes. Just changing the extension title and icon should be sufficient to improve discoverability.

mnxn avatar Nov 30 '20 21:11 mnxn

That extension is still listed under hackwaly.ocaml:

That's very strange: https://github.com/hackwaly/vscode-ocaml/pull/124/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R3

I don't see how it would be possible for the vsce tool to detect an extension ID

Maybe it cross-checks with the VCS URI?

I'd prefer to wait until then to change the extension ID in order to minimize changes

Sure that sounds reasonable

tmattio avatar Nov 30 '20 21:11 tmattio

That extension is still listed under hackwaly.ocaml: https://marketplace.visualstudio.com/items?itemName=hackwaly.ocaml

hackwaly.ocaml-deprecated is a 404: https://marketplace.visualstudio.com/items?itemName=hackwaly.ocaml-deprecated

I don't see how it would be possible for the vsce tool to detect an extension ID change since it reads from the package.json to decide what extension it's publishing in the first place.

If we're planning to move to the ocaml organization/publisher (are we?), I'd prefer to wait until then to change the extension ID in order to minimize changes. Just changing the extension title and icon should be sufficient to improve discoverability.

I worry about directly change ocaml to ocaml-deprecated in package.json just results two published extensions - one is ocaml the other is ocaml-deprecated. So I didn't use ocaml-deprecated to publish.

AFAIK. There's no name conflict policy in vscode marketplace. So you can use ocamllabs/ocaml without issue.

EDIT: There may be some vsce command to rename extension. Not just change the package.json. But I have no time to find out that.

hackwaly avatar Dec 01 '20 05:12 hackwaly

I worry about directly change ocaml to ocaml-deprecated in package.json just results two published extensions - one is ocaml the other is ocaml-deprecated. So I didn't use ocaml-deprecated to publish.

That is my guess too. That is also why I don't want to rename ocamllabs.ocaml-platform to ocamllabs.ocaml right now, because it might confuse users.

AFAIK. There's no name conflict policy in vscode marketplace. So you can use ocamllabs/ocaml without issue.

Yeah, I don't think there's any name conflict right now. My worry is about the user experience of switching the ID: if it causes two published extension or stops the extension from updating.

mnxn avatar Dec 01 '20 05:12 mnxn

I don't think we should wait for the move of the repository to merge this anymore, so I reverted the name change, and this PR now only updates the displayed name and the logo.

Ready to merge when approved.

tmattio avatar Jan 20 '21 10:01 tmattio

The extension is still being featured on the extensions page: https://marketplace.visualstudio.com/vscode Should we wait to change the display name/logo until it stops being featured?

@smorimoto: any thoughts?

mnxn avatar Jan 20 '21 21:01 mnxn

There should be no problem to change the logo, but I don't know the name. If it can be handled as another extension, you should be very careful about that. If something goes wrong, there's nothing to gain and everything to lose.

smorimoto avatar Jan 22 '21 02:01 smorimoto

Honestly, I don't really think icon and name are blocking adoption right now. In the case of F#, Ionide is the most popular one, but I have never decided not to use it by name, and no one around me has made such a decision. Maybe the best way is to promote it enough in a diffusive medium like OCaml's Twitter account. I doubt the logo and name are really a problem.

smorimoto avatar Jan 22 '21 02:01 smorimoto

Closing for now, as it seems a bit stale. I guess we can revisit when (if) we move the repo to the OCaml org.

tmattio avatar Feb 08 '21 11:02 tmattio

Re-opening the PR - let's coordinate the rebranding and move to the OCaml organisation.

What's blocking the transfer of the repository to ocaml/vscode-ocaml is an announcement on Discuss. I'm writing it, but it'd be great if I could get approval on principle for this PR, then I could:

  • Post the announcement
  • Move the repository
  • Cut a release

Moving the extension to an official publish for OCaml can be done separately and will need assistance from the VSCode support, so let's do this as a second step.

tmattio avatar Sep 14 '22 10:09 tmattio