Re: [Apache Bloodhound] #146: Inline editing of objects

2013-01-02 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  gjm
  Type:  enhancement  | Status:  review
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+

Comment (by gjm):

 r1427698 applies t146_r1418195_ticket_header.diff

 r1427699 applies t146_r1420132_inplace_ticket_fields.diff

 r1427702 applies t146_r1420132_inplace_ticket_desc.diff

 r1427717 applies t146_r1420132_inplace_ticket_form.diff

 thanks olemis.

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:41
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2013-01-02 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  gjm
  Type:  enhancement  | Status:  closed
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:  fixed|   Keywords:  inplace bheditable
--+
Changes (by gjm):

 * status:  review = closed
 * resolution:   = fixed


Comment:

 Calling this complete - further enhancements in new tickets.

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:42
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-12-21 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  gjm
  Type:  enhancement  | Status:  review
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+

Comment (by gjm):

 This looks pretty good for committing. There are a few issues left but I
 think it is probably better to commit this work and raise new tickets for
 any required tweaks.

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:39
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-12-21 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  gjm
  Type:  enhancement  | Status:  review
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+

Comment (by olemis):

 Replying to [comment:39 gjm]:
 [...]
  There are a few issues left

 I've detected some potential issues too ... low priority afaics .

  but I think it is probably better to commit this work and raise new
 tickets for any required tweaks.

 I'm ok with that approach .

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:40
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-12-17 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  gjm
  Type:  enhancement  | Status:  review
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+
Changes (by jdreimann):

 * owner:  jdreimann = gjm


Comment:

 I assume the textfield next to the Update button is the ticket summary?
 I would change the button style from btn-primary to btn-warning or btn-
 danger for higher contrast, but other than that I'd say +1 to commit this
 so we can actually run it locally and improve on what's built already.
 This ticket has been going on for too long and unwieldy.

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:37
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-12-15 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  jdreimann
  Type:  enhancement  | Status:  review
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+
Changes (by olemis):

 * owner:  olemis = jdreimann
 * status:  accepted = review


Comment:

 After comment:32 I have refreshed previous patches and attached new patch
 implementing ticket submission . Patch order in comment:27 updated
 accordingly . Other minor enhancements have been added consisting of :

 summary: BH Theme #146 : In-place submit form . Sticky panel relocated
 and other minor changes
 summary: BH Theme #146 : Discard inline modifications
 summary: BH Theme #146 : Tooltips for workflow actions
 summary: BH Theme #146 : Edit mode not reentrant plus .editing =
 .edit-active
 summary: BH Theme #146 : Workflow drop down visible on affix
 summary: BH Theme #146 : Use JS templates to render in-place submit
 controls
 summary: BH Theme #146 : Ticket in-plece submit controls

 Anyway , after all this editable state will look like shown below :

 [[Image(bh_theme_x_91_ticket_responsive_submit_workflow.png, width=600)]]

 I'm forwarding ticket for further review of workflow drop down menu from a
 UI/UX perspective .

 A few comments on that subject .

   1. Update / cancel buttons placed in sticky panel so that they'll
 always be handy in edit mode
   2. Update button highlighted (`btn-primary`) because it has to
 be that relevant on edit mode . Otherwise it's hard to notice
 since it shows up out of nowhere .
   3. Reduced space = reduced visibility of action hints (i.e. tooltips)
   4. Selected workflow action appended to ''Update'' label to allow
 one click edits . Otherwise it is obscured the intentions and
 difficult
 to figure out what's gonna happen with ticket in the end .

 Beyond this , there should be some room for enhancements . That's sure . I
 suggest to tackle improvements on separate tickets .

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:36
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-12-14 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+

Comment (by olemis):

 Replying to [comment:33 jdreimann]:
  Replying to [comment:32 olemis]:
   [...] Anyway , after all this editable state will look like shown
 below :
  
   [img]
 
  I assume the look of the drop down button and the red background for
 selected radio button/checkbox are browser dependent?

 Browser dependent. Would be nice to have consistent look and feel across
 different platforms .

  Just checking to make sure it isn't styled that way.

 ;)

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:34
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-12-11 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+

Comment (by olemis):

 I have submitted new files to rebase patches against r1420132 . Patch
 order in comment:27 updated accordingly . I have also added a new patch to
 edit special ticket fields , namely description, keywords, summary and
 reporter . At present they will remain in view state if permissions are
 not sufficient . Other minor enhancements have been added like tagging
 editable fields with `data-edit=inplace` , custom textarea size , etc
 ...

 Anyway , after all this editable state will look like shown below :

 [[Image(bh_theme_x_90_ticket_responsive_edit_all.png, width=600)]]

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:32
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-12-10 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+

Comment (by gjm):

 Note: t146/t146_r1418195_scrollspy_btn.diff committed as part of #295

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:31
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-12-07 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+

Comment (by olemis):

 The new approach to in-place editing in ticket view is not compatible with
 jEditable afaics . Hence I've built everything from scratch . Attached
 patches against theme plugin add the following features :

   1. Enhanced ticket fields list updated according to responsive mockups
   2. Enhanced scroll spy nav bar implemented with Bootstrap button groups
  like sketched in responsive mockups
   3. Modify ticket section is gone
   4. On clicking Modify Ticket button in scroll spy nav bar , ticket view
  enters editable state (suggested by jure)

 Patches should be applied against bloodhound_theme @ r1418195 in the
 following order :

 {{{
 #!sh

 $ hg qapplied
 t146/t146_r1418195_scrollspy_btn.diff
 t146/t146_r1418195_ticket_header.diff
 t146/t146_r1418195_inplace_ticket_fields.diff

 }}}

 Issues found so far :

   1. In editable state scroll spy does not activate nav item accurately.
  This happens because heights are different (e.g. wiki toolbar added
  on top of some textarea elements , etc ...) to the values when
  scroll spy was installed.

 ... /me figuring out how to fix .

 After all this ticket view will look like shown below in view mode ...

 [[Image(bh_theme_x_86_ticket_responsive_view_state.png, width=600)]]

 ... and in editable mode ...

 [[Image(bh_theme_x_87_ticket_responsive_edit_state.png, width=600)]]

 Still there are many open questions :

   1. What happens after ticket enters editable state ?
   2. What about workflow actions now that Modify Ticket section is gone?
   3. Should description, keywords and summary be put into editable
  as well (... IMO they should , I'm just looking for
  confirmation ...) ?

 Feedback is welcome .

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:27
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-12-07 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+

Comment (by gjm):

 Replying to [comment:27 olemis]:
1. What happens after ticket enters editable state ?
 after edits we clearly need a Submit Changes button and a cancel button.
 Submission and cancelling probably imply a change back to the non-editable
 state so perhaps we need a revert button that only resets the fields as
 well?
2. What about workflow actions now that Modify Ticket section is gone?
 We still need the modify ticket area for that until there is an
 alternative solution for workflow.
3. Should description, keywords and summary be put into editable
   as well (... IMO they should , I'm just looking for
   confirmation ...) ?
 Yes.. until otherwise noted, we will require that.

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:28
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-12-07 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+

Comment (by jdreimann):

 Replying to [comment:28 gjm]:
  Replying to [comment:27 olemis]:
 1. What happens after ticket enters editable state ?
  after edits we clearly need a Submit Changes button and a cancel button.
 Submission and cancelling probably imply a change back to the non-editable
 state so perhaps we need a revert button that only resets the fields as
 well?

 I think Submit button and a Cancel button (with `btn-link` class to show
 it's a secondary action) will be enough.

 2. What about workflow actions now that Modify Ticket section is
 gone?
  We still need the modify ticket area for that until there is an
 alternative solution for workflow.

 What stops us from workflow simply being a dropdown of options? `review,
 resolve, info request` etc.

 3. Should description, keywords and summary be put into editable
as well (... IMO they should , I'm just looking for
confirmation ...) ?
  Yes.. until otherwise noted, we will require that.
 +1

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:29
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-12-07 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+

Comment (by gjm):

 Replying to [comment:29 jdreimann]:
  Replying to [comment:28 gjm]:
   Replying to [comment:27 olemis]:
  1. What happens after ticket enters editable state ?
   after edits we clearly need a Submit Changes button and a cancel
 button. Submission and cancelling probably imply a change back to the non-
 editable state so perhaps we need a revert button that only resets the
 fields as well?
 
  I think Submit button and a Cancel button (with `btn-link` class to show
 it's a secondary action) will be enough.

 Fair enough.. certainly good enough for now

 
  2. What about workflow actions now that Modify Ticket section is
 gone?
   We still need the modify ticket area for that until there is an
 alternative solution for workflow.
 
  What stops us from workflow simply being a dropdown of options? `review,
 resolve, info request` etc.

 I was thinking a similar way but the control is a little more complicated
 in needing a second input to change assignment for some actions. I don't
 know if there would be more cases to take into account with plugins that
 extend the functionality of course.

  3. Should description, keywords and summary be put into editable
 as well (... IMO they should , I'm just looking for
 confirmation ...) ?
   Yes.. until otherwise noted, we will require that.
  +1

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:30
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-12-02 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:
  Type:  enhancement  | Status:  assigned
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+
Changes (by olemis):

 * status:  needinfo = assigned


Comment:

 Instructions confirmed in [http://mail-archives.apache.org/mod_mbox
 /incubator-bloodhound-dev/201211.mbox/%3cCAKoya91u=peYCkfxXJ4LgZ-
 FUV6yJ5zt-_Q=hcd3ynmrrfg...@mail.gmail.com%3e a message] sent to
 bloodhound-dev ML .

 I'll start working on this following suggestions provided by gjm and
 jdreimann .

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:25
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-12-02 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 4
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+
Changes (by olemis):

 * status:  assigned = accepted
 * owner:   = olemis


-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:26
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-11-04 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:
  Type:  enhancement  | Status:  needinfo
  Priority:  critical |  Milestone:  Release 3
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+
Changes (by olemis):

 * status:  accepted = needinfo
 * owner:  olemis =


Comment:

 Work on this ticket is suspended until we figure out how in-place editing
 should work . Open discussion's been held in bloodhound-
 d...@incubator.apache.org

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:24
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-10-08 Thread Apache Bloodhound
#146: Inline editing of objects
--+
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:  inplace bheditable
--+

Comment (by olemis):

 Attached patches have to be refreshed after #217 , #223 and especially
 considering [comment:9:ticket:206 enhancements] in ticket fields UI .

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:23
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-10-04 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

Comment (by jdreimann):

 Replying to [comment:19 olemis]:
Inplace editor for ticket radio fields . Please take a look at image
 below .
Feedback is welcome

 Looks good to me.

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:21
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-09-30 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

Comment (by olemis):

 I have refreshed and uploaded two patches . I've also erased patches
 introducing iPhone style checkboxes from my local patch queue . Hence I've
 also updated [comment:14:ticket:146 patch order] . Main changes are :

   - Use plain old checkboxes
   - Inplace editor for ticket radio fields . Please take a look at image
 below .
 Feedback is welcome

 [[Image(bh_theme_x_71_ticket_inplace_radio.png, width=600)]]

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:19
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-09-28 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

Comment (by jdreimann):

 Replying to [comment:16 olemis]:
  This is closer to completion now ... at least the frontend is almost
 ready . This is what iPhone checkboxes look like . Feedback is welcome .

 Where does the requirement for iPhone checkboxes come from? I don't
 remember this being discussed anywhere. If you want to implement them,
 please propose that in a separate ticket. In this ticket, please focus on
 enabling inline edits.

 For now I'd appreciate if you would remove them from proposed patches
 because they are inconsistent with the current design and were never
 designed for mouse-operation. Not even Apple uses them in their desktop
 software, apart from a handful exceptions.

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:17
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-09-28 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

Comment (by olemis):

 Replying to [comment:17 jdreimann]:
  Replying to [comment:16 olemis]:
   This is closer to completion now ... at least the frontend is almost
 ready . This is what iPhone checkboxes look like . Feedback is welcome .
 
  Where does the requirement for iPhone checkboxes come from? I don't
 remember this being discussed anywhere.

 just a suggestion ...

  If you want to implement them, please propose that in a separate ticket.
 In this ticket, please focus on enabling inline edits.
 

 ... so if this is not correct I revert those changes and use plain-old
 checkboxes . No big deal .

 Now I ask , what inplace radio ticket fields editor will look like ?

  For now I'd appreciate if you would remove them from proposed patches
 because they are inconsistent with the current design and were never
 designed for mouse-operation.

 ok

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:18
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-09-26 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

Comment (by olemis):

 After a lot of fighting against external js libs , I think I won the first
 battle and managed to implement in place editor for ticket checkbox fields
 . I have refreshed patches accordingly and updated application order
 mentioned before in comment:14 . Only frontend has been improved so it's
 still work in progress at the moment , as the backend still needs to be
 developed .

 New features described below .

  Dashboard plugin 

   - [http://github.com/tdreyno/iphone-style-checkboxes iPhone style
 checkboxes js library] added .
 ''MIT license''
   - Modifications for ''jEditable'' .
   - Architecture of Bloodhound editable data API modified to support
 references to functions
 in `data-edit*` attributes (by using `javascript:` prefix) .
   - iPhone checkbox jEditable type ... some minor enhancements still
 needed .

  Theme plugin 

   - Enhanced code to load values for select and checkbox fields
   - Changes for iPhone checkbox editable type propagated to render these
 controls in
 ticket view .

 This is closer to completion now ... at least the frontend is almost ready
 . This is what iPhone checkboxes look like . Feedback is welcome .

 [[Image(bh_theme_x_70_ticket_inplace_iphone_cb.png, width=600)]]

 Only editors for radio ticket type are missing ... coming soon .

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:16
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-09-21 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

Comment (by olemis):

 Replying to [comment:11 jdreimann]:
  Replying to [comment:10 olemis]:
   The following screenshot illustrates what it looks like . Feedback is
 welcome .
 
  I assume the button with the check mark is to confirm the selection and
 'commit' it?

 yes

  This should not be required.

 ok. I'll remove that for select fields .

  Part of the case for inline editing is that confirmation is only
 required when
  1. It's difficult (time, effort) or impossible to revert fully.
  1. Automatically saved information would be incomplete.
 
  An example for 1. is when a ticket is when stuff gets deleted. An
 example for 2. is changing a ticket description, which you probably
 wouldn't want to change for everyone before you're done typing.
 

 IMO submit button does need to be required for all kinds of text fields
 (i.e. raw wiki textarea, raw and wiki text field ) . Consider the case
 when user starts typing but suddenly decides it's all wrong . Hence there
 should be a way to revert in place edits (e.g. Submit + Cancel button ?)

  I think we'll get closer to a proper definition by trial and error, but
 I think our default approach should be to not ask for confirmation unless
 in cases of grave danger.

 ... and when there's a need to cancel / undo in place modifications , like
 is the case for text fields .

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:12
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-09-21 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

Comment (by jdreimann):

 Replying to [comment:12 olemis]:
  IMO submit button does need to be required for all kinds of text fields
 and
  [..] there's a need to cancel / undo in place modifications , like is
 the case for text fields .
 Yep, that's what I was trying to say :)
 +1

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:13
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-09-08 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  olemis
  Type:  enhancement  | Status:  accepted
  Priority:  critical |  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---
Changes (by olemis):

 * status:  new = accepted
 * owner:  nobody = olemis


-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:9
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-08-29 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  nobody
  Type:  enhancement  | Status:  new
  Priority:  critical |  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---
Changes (by jdreimann):

 * priority:  major = critical


-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:7
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-08-29 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  nobody
  Type:  enhancement  | Status:  new
  Priority:  critical |  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

Comment (by gjm):

 Replying to [comment:4 olemis]:
 
  The way I see it we have three options :
 
1. Use ''jEditable'' except auto-grow plugin.
  - We'd have to decide whether this is a crucial feature

 OK, go with option 1. We can at least see if the autogrow functionality is
 missed

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:8
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-08-21 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  nobody
  Type:  enhancement  | Status:  new
  Priority:  major|  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

Comment (by jdreimann):

 Replying to [comment:4 olemis]:
  [...] If ''Modify'' / ''Change'' will disappear then what is supposed to
 happen when ''Javascript'' is disabled ?

 How about they stay until Javascript removes them? Or what would you
 suggest?

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:5
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-08-21 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  nobody
  Type:  enhancement  | Status:  new
  Priority:  major|  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

Comment (by olemis):

 Replying to [comment:5 jdreimann]:
  Replying to [comment:4 olemis]:
   [...] If ''Modify'' / ''Change'' will disappear then what is supposed
 to happen when ''Javascript'' is disabled ?
 
  How about they stay until Javascript removes them?

 That's ok afaics.

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:6
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-08-20 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  nobody
  Type:  enhancement  | Status:  new
  Priority:  major|  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

Comment (by jdreimann):

 Am I right in thinking that the licensing question regarding jEditable
 [http://mail-archives.apache.org/mod_mbox/incubator-bloodhound-
 dev/201203.mbox/%3C4F61C392.7060403%40wandisco.com%3E raised by Gary
 previously] hasn't been resolved yet?

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:3
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-08-20 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  nobody
  Type:  enhancement  | Status:  new
  Priority:  major|  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

Comment (by olemis):

 The way I see it we have three options :

   1. Use ''jEditable'' except auto-grow plugin.
 - We'd have to decide whether this is a crucial feature
   2. Use ''jEditable'' and implement a replacement
  for auto-grow functionality
 - We'd have to assess whether this is a feasible task
   3. Do not use ''jEditable'' at all, and choose
  a similar library
 - So far I have not found something as complete
   as ''jEditable'' , especially supporting integrate of
   the editor with server-side WikiFormatting renderer .
 - I'm not an expert and I have not exhausted all
   the possibilities

 Beyond that , there is something else we need to discuss . If ''Modify'' /
 ''Change'' will disappear then what is supposed to happen when
 ''Javascript'' is disabled ?

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:4
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects

2012-08-13 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  nobody
  Type:  enhancement  | Status:  new
  Priority:  major|  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

Comment (by olemis):

 So what shall we do about this ticket ?
 Is it ok to use ''jEditable'' for this ?
 If the decision is to move forward with another library then which one ?

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:2
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker


Re: [Apache Bloodhound] #146: Inline editing of objects (was: In-place editing of objects)

2012-07-17 Thread Apache Bloodhound
#146: Inline editing of objects
--+---
  Reporter:  jdreimann|  Owner:  nobody
  Type:  enhancement  | Status:  new
  Priority:  major|  Milestone:  Release 2
 Component:  dashboard|Version:
Resolution:   |   Keywords:
--+---

-- 
Ticket URL: https://issues.apache.org/bloodhound/ticket/146#comment:1
Apache Bloodhound https://issues.apache.org/bloodhound/
The Apache Bloodhound (incubating) issue tracker