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

added ssi and xsl file extensions #7210

Merged
merged 2 commits into from
Mar 20, 2014
Merged

Conversation

gidsg
Copy link
Contributor

@gidsg gidsg commented Mar 16, 2014

Added SSI as a file extension of HTML and XSL as a file extension of XML.

@ingorichter ingorichter assigned RaymondLim and redmunds and unassigned RaymondLim Mar 18, 2014
@redmunds
Copy link
Contributor

I'm OK with adding the .xsl file extension to XML language, but .ssi is too generic to add to HTML language, so please remove that change.

You can write a simple extension to add the .ssi file extension to HTML as follows:

Inside Brackets, use Help > Show Extensions Folder to open the extensions folder in OS file system. Then drag that folder back onto Brackets to open folder as your current project. Then open the /user folder and create a folder for your extension called something like /add-ssi.

Create a file named main.js in the /add-ssi folder that looks something like this:

/*jslint vars: true, plusplus: true, devel: true, nomen: true, regexp: true, indent: 4, maxerr: 50 */
/*global define, $, brackets, window */

define(function (require, exports, module) {
    "use strict";

    var LanguageManager = brackets.getModule("language/LanguageManager");
    var language = LanguageManager.getLanguage("html");
    language.addFileExtension("ssi");
});

Then shutdown/restart Brackets (or use Debug > Reload With Extensions) to load your extension.

@gidsg
Copy link
Contributor Author

gidsg commented Mar 19, 2014

Thanks @redmunds I have made the requested change.

@redmunds
Copy link
Contributor

Looks good. Thanks. Merging.

redmunds added a commit that referenced this pull request Mar 20, 2014
added ssi and xsl file extensions
@redmunds redmunds merged commit ee8d3bf into adobe:master Mar 20, 2014
@gidsg gidsg deleted the add-file-extensions branch June 23, 2014 11:00
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.

3 participants