-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Issues with keybindings manager #1495
Comments
Hi--thanks for filing this! For (3), submenus are unfortunately not implemented yet, but they're on our backlog: https://trello.com/c/OrIaeTsS -- though we might get to native menus/submenus first: https://trello.com/c/EwLGRkYe For (5), it's okay for now to go ahead and use the _codeMirror property for things that aren't directly exposed in Editor, with the understanding that these might get wrapped in some other API in the future. For (2) and (4), could you create separate issues for these and make this issue just be about (1)? We'd like to track these different issues separately. Thanks! |
Removing sprint 14 label |
(1) is a fact of life with the version of RequireJS we're using, but it looks like it might be fixed in the current version. If you're feeling especially motivated, you could try upgrading Brackets to RequireJS 2.x to see if that's true :-) |
We're now on require.js 2.1.1 (#1968) |
@DennisKehrig does Sprint 15 contains rjs 2.1.1? |
@sergeche No, seems like the upgrade was done shortly after releasing sprint 15. Sprint 16 is just around the corner, though. Until then, you can use the latest code as described here: https://github.com/adobe/brackets/wiki/How-to-Hack-on-Brackets |
Current master branch fails to load extensions here: https://github.com/adobe/brackets/blob/master/src/utils/ExtensionLoader.js#L59 Anyway, I was able to fix some of my core modules and create custom builder that concats and obfuscates a whole plugin into a single The plugin is available here: https://github.com/sergeche/zen-coding/downloads If there’s no errors, I can announce it on http://docs.emmet.io and twitter. |
There were some changes to the paths being used for extensions... and maybe a shell update is needed to make these work (my extensions aren't being loaded either at the moment). Unfortunately I don't have access to a more recent shell right now. |
We introduced getActiveEditor which I recommend using over getFocusedEditor. |
OK, thanks, I’ll for your review. |
Using Sprint 16 (not yet publicly available) it works great right out of the box! Both on the Mac and on Windows. I just get a few duplicated shortcuts:
And this:
Brackets isn't yet using Node.js, that's only the case for the in-browser branch. The Chromium based shell provides access to the native file system with a custom API, exposed via brackets.fs. A Node.js based version would wrap node's file system API accordingly. I'm not sure it's possible to add synchronous functions to the shell, but it might be. Does it need to be synchronous? |
Thanks :) I’ve updated and re-uploaded Brackets plugin: no more file-based actions and changed keymap so there should be no duplicates. As for endless loop error, it’s a protection against invalid abbreviations (or buggy parser). Can you post an abbreviation that you’re trying to expand that leads to this error? As for sync file reader: Emmet currently uses sync calls in actions. I think I can switch to async pattern, but a bit later. So, I think Brackets plugin is stable and can be announced :) |
@sergeche: that's awesome! If you want to do the honors of announcing it yourself, I'd suggest this:
|
@peterflynn I would really appreciate, if Adobe Team announce it :) I can wait for String 16 to make final tests. |
@sergeche: We haven't done the official announcement yet, but the build is now done and posted here: https://github.com/adobe/brackets/downloads |
@peterflynn just checked with Sprint 16: everything works fine. I’ve slightly updated keymap and re-uploaded plugin to https://github.com/sergeche/zen-coding/downloads |
Closing this since issue (1) here is now fixed, and the others have either been spun off as bugs or recorded in the backlog. @sergeche, we have a lot of users who are excited to start using your extension now! |
Just a quick note: I moved repo to new location: https://github.com/emmetio/emmet |
Hey @sergeche, you're giving me an incentive to support ZIP files in the extension manager. ;) Yours is the first that is not directly installable via git clone. Since I just saw the download description - "Emmet for Adobe Brackets v1.0 (beta)": I recently read that apparently we try to not refer to Brackets as "Adobe Brackets", but just Brackets. The idea is that the project should stand on its own. Of course you can do with that piece of information whatever you want. :) |
That was quick! Thanks! |
@DennisKehrig no problem, just renamed the plugin: https://github.com/emmetio/emmet/downloads |
@sergeche Re: infinite recursion - just select an opening HTML tag, like |
Oh, it breaks code indentation as well. I’m fixing it right now. |
Fixed this bug and re-uploaded plugin. |
Hey Sergey, you can replace the call to
Is that good enough for you? Thanks, Dennis |
@njx: Assigning to myself to keep track of it |
Thanks! |
@DennisKehrig I’ve pushed update that conditionally uses shared Tab handler. I’m not updating plugin since it’s useless for now, I guess :) |
Sounds like a safe choice. :) |
@sergeche Alright, I talked to the team. If you could just add a link to the license in the comment above the function, we'd appreciate it a lot! Here's a suggestion:
Thank you, and thanks a lot for creating the plugin in the first place! |
Hi guys,
I just made initial implementation of Emmet toolkit (ex-Zen Coding): https://github.com/sergeche/zen-coding
The plugin can be downloaded here: http://media.chikuyonok.ru/Emmet-Brackets.zip
I have a few issues with this plugin implementation:
require
function. Emmet toolkit itself usesrequire
method of core object (zen_coding.require
) to get internal modules. But Brackets substitutes everything that named asrequire
with its own bundle loader so I had to manually rename this method to something else to make core work.getOption()
method so I had to use_codeMirror
property directly to get access for current editor’s syntax and preferences.The text was updated successfully, but these errors were encountered: