cfjedimaster / brackets-jshint

Adds JSHint support to Brackets
MIT License
131 stars 41 forks source link

Bootstrap Update presumably in Sprint 26 #4

Closed WebsiteDeveloper closed 10 years ago

WebsiteDeveloper commented 11 years ago

For sprint 26 you will have to update the table classes because bootstrap 2.3.1 will be pulled in see https://github.com/adobe/brackets/pull/3672 If you like i could also submit a pull request once the update landed.

cfjedimaster commented 11 years ago

If you know the fix, I certainly wouldn't mind a pull request. I'd appreciate it.

On Fri, May 17, 2013 at 12:00 PM, Bernhard Sirlinger < notifications@github.com> wrote:

For sprint 26 you will have to update the table classes because bootstrap 2.3.1 will be pulled in see adobe/brackets#3672https://github.com/adobe/brackets/issues/3672 If you like i could also submit a pull request once the update landed.

— Reply to this email directly or view it on GitHubhttps://github.com/cfjedimaster/brackets-jshint/issues/4 .

Raymond Camden, Adobe Developer Evangelist

Email : raymondcamden@gmail.com Blog : www.raymondcamden.com Twitter: cfjedimaster

redmunds commented 11 years ago

The fixes for tables are:

Note that jQuery, Bootstrap, and LESS migration code is now in master to be released in Sprint 26.

On another issue, while playing around with this extension, I can't seem to generate any errors, no matter what random junk I put in the file. Same results in Sprint 25, so it does not seem to be related to Bootstrap migration. Maybe my junk is not random enough?

cfjedimaster commented 11 years ago

Looks to be a bug. Working on it now.

cfjedimaster commented 11 years ago

Oh yeah - I screwed up when I added support for project-local config files. Should be a quick fix - but I'm about to fly to Scotland. May be a few days.

cfjedimaster commented 10 years ago

Closing.