Skip to content

Conversation

ShahparShabani
Copy link
Contributor

Tasks:

  • added delete icon
  • got feedback

Informations:

Q A
Bug fix? yes
New feature? no
Tests pass? none
Fixed tickets none
Doc none

@wachterjohannes
Copy link
Member

@@ -221,6 +221,7 @@ define([

// list-toolbar template
listTemplate = function() {
this.console.log("exxxxxiiiit");
Copy link
Contributor

Choose a reason for hiding this comment

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

There should be no console.log in the final code

@danrot
Copy link
Contributor

danrot commented Jan 7, 2015

Is this PR still needed?

@wachterjohannes
Copy link
Member

@danrot i think we can close this PR nothing happens since 10.Nov!

@danrot danrot closed this Feb 24, 2015
@danrot danrot deleted the feature/add-deletetoolbar-icon branch February 24, 2015 13:07
alexander-schranz pushed a commit to alexander-schranz/sulu that referenced this pull request Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants