Ticket #2117 (closed bug: obsolete)

Opened 10 years ago

Last modified 8 years ago

gradient-issues - Putting the two ends of a gradient line one over another throws an exception.

Reported by: milen Owned by:
Priority: major Milestone: M12_RELEASE
Component: uncategorized Version: 2.0
Keywords: Cc:
Category: unknown Effort:
Importance: 69 Ticket_group:
Estimated Number of Hours: 0 Add Hours to Ticket: 0
Billable?: yes Total Hours: 0
Analysis_owners: milen Design_owners: diana
Imp._owners: diana Test_owners:
Analysis_reviewers: deyan, pap Changelog: Changelog
Design_reviewers: pap Imp._reviewers:
Test_reviewers: Analysis_score: 0
Design_score: 3.5 Imp._score: 4
Test_score: 0

Description (last modified by todor) (diff)

  1. Create a new book.
  2. Open the appearance HUD of a page.
  3. Select gradient from dropdown in the background area.
  4. Open the gradient HUD.
  5. Put the two gradient circles one over another.

As a result exception is thrown (division by 0).

Attachments

gradients.txt (28.7 KB) - added by milen 10 years ago.
2117.patch (2.3 KB) - added by diana 10 years ago.

Change History

Changed 10 years ago by milen

comment:1 Changed 10 years ago by milen

  • Status changed from new to s1b_analysis_finished

comment:2 Changed 10 years ago by deyan

  • Importance set to 69
  • Summary changed from Putting the two ends of a gradient line one over another throws an exception to gradient-issues Putting the two ends of a gradient line one over another throws an exception; gradient should be streched

Batch update from file report_1.csv

comment:3 Changed 10 years ago by todor

  • Summary changed from gradient-issues Putting the two ends of a gradient line one over another throws an exception; gradient should be streched to gradient-issues - Putting the two ends of a gradient line one over another throws an exception; gradient should be streched

comment:4 Changed 10 years ago by todor

  • Description modified (diff)

comment:5 Changed 10 years ago by todor

  • Summary changed from gradient-issues - Putting the two ends of a gradient line one over another throws an exception; gradient should be streched to gradient-issues - Putting the two ends of a gradient line one over another throws an exception.

comment:6 Changed 10 years ago by deyan

  • Description modified (diff)

Batch update from file active_tickets.csv

comment:7 Changed 10 years ago by todor

  • Status changed from s1b_analysis_finished to closed
  • Resolution set to later
  • Description modified (diff)
  • Changelog set to [wiki:Changelog]

comment:8 Changed 10 years ago by diana

  • Design_owners set to diana
  • Resolution later deleted
  • Status changed from closed to new
  • Imp._owners set to diana

comment:9 Changed 10 years ago by diana

  • Status changed from new to s1b_analysis_finished

comment:10 Changed 10 years ago by diana

  • Status changed from s1b_analysis_finished to s2a_design_started
  • In org.sophie2.main.app.halos.huds.gradient package create new class SophieGradientPreviewPanel extends GradientPreviewPanel - Class for the gradient preview. Used only to catch the exception from the GradientPreviewPanel#getGradient() method when the two points are equal. This class has private MultipleGradientPaint lastGradient - a variable to hold the last gradient gained by the gtGradient function.
  • In SophieGradientPreviewPanel override the getGradient method, catch the exception from the GradientPreviewPanel class and return the lastGradient variable.
  • In GradientPicker:gradientPreview create gradient preview panel instance from SophieGradientPreviewPanel class.

Changed 10 years ago by diana

comment:11 Changed 10 years ago by diana

  • Status changed from s2a_design_started to s3b_implementation_finished

comment:12 Changed 10 years ago by pap

  • Status changed from s3b_implementation_finished to s3c_implementation_ok
  • Design_score changed from 0 to 3.5
  • Design_reviewers set to pap
  • Analysis_reviewers set to deyan, pap
  • Imp._score changed from 0 to 4
  • Commited in [9242].
  • It is not nice that we have to swallow the exception this way.

comment:13 Changed 8 years ago by meddle

  • Status changed from s3c_implementation_ok to closed
  • Resolution set to obsolete

Closing all the tickets before M Y1

Note: See TracTickets for help on using tickets.