Skip to content
This repository was archived by the owner on Nov 19, 2019. It is now read-only.

Update sidebar.coffee #40

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update sidebar.coffee #40

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions assets/js/directives/sidebar.coffee
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -206,7 +206,7 @@ module.directive "plunkerSidebar", [ "$timeout", "$q", "session", "notifier", "v
$scope.session = session
$scope.visitor = visitor
$scope.promptFileAdd = ->
notifier.prompt "New filename", "",
notifier.prompt "New filename", " (supported formats include: .html,.md,.js,.jade,.less,.coffee)",
Copy link
Member

@ggoodman ggoodman Jul 8, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is a good idea in theory, but I do not want to mislead users into thinking that these are the only formats supported.

In reality, any text-based format can be created and requested without transformation. It is true the Plunker only supports transforming a limited number of file types.

What do you think?

Copy link
Author

@ImaginaryDevelopment ImaginaryDevelopment Jul 9, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that I'd love somewhere on the UI directly to know what types support transformation as I generally try to google around everytime I can't remember what is supported. I thought maybe right on the prompt would be nice, but perhaps more text would fit there or on a title/tooltip on the new button?

confirm: (filename) -> session.addBuffer(filename, "", activate: true)

$scope.revertTo = (rel) ->
Expand Down Expand Up @@ -251,4 +251,4 @@ module.directive "plunkerSidebar", [ "$timeout", "$q", "session", "notifier", "v
$scope.addthis = true

dereg()
]
]