Ticket #2327 (closed bug: obsolete)

Opened 11 years ago

Last modified 9 years ago

saving over page template saves second page template

Reported by: danvisel Owned by:
Priority: minor Milestone: X3
Component: uncategorized Version: 2.0
Keywords: Cc:
Category: unknown Effort:
Importance: Ticket_group:
Estimated Number of Hours: 0 Add Hours to Ticket: 0
Billable?: yes Total Hours: 0
Analysis_owners: danvisel Design_owners: deni
Imp._owners: deni Test_owners:
Analysis_reviewers: deni Changelog: Changelog
Design_reviewers: Imp._reviewers:
Test_reviewers: Analysis_score: 4.5
Design_score: 4 Imp._score: 4
Test_score: 0

Description (last modified by deni) (diff)

  1. Open Sophie.
  2. Create a new book.
  3. Open the page appearance HUD; color the page red.
  4. Save the page as a page template named "my page template".
  5. Add a second page to the book, and go to this page.
  6. Open the page appearance HUD; color the page blue.
  7. Save the page as a page template named "my page template".

On 7, you'll be asked if you want to overwrite the existing page template with that name. Say yes. Then look in the page templates palette: there are two page templates named "my page template". Go back to page 1: it's still red. (Based on the warning, we would expect it to have turned blue.)

Both of the page template entries in the page template palette are the re-saved version: both turn a page blue if you apply them to a page.

Resaving a page template should result in only one page template in the page template palette; it should also apply the new page template to all pages currently using that page template.

Attachments

overriding-templates.patch (1.2 KB) - added by deni 10 years ago.

Change History

comment:1 Changed 11 years ago by danvisel

  • Status changed from new to s1b_analysis_finished

comment:2 Changed 10 years ago by deyan

  • Description modified (diff)

Batch update from file active_tickets.csv

comment:3 Changed 10 years ago by deni

  • Design_owners set to deni
  • Status changed from s1b_analysis_finished to s2a_design_started
  • Imp._owners set to deni
  • Description modified (diff)
  • Changelog set to [wiki:Changelog]
  • Analysis_score changed from 0 to 4.5
  • Analysis_reviewers set to deni
  • Clear and detailed explanation.

comment:4 Changed 10 years ago by deni

  • Status changed from s2a_design_started to s3b_implementation_finished
  • Templates appear twice in the palette:
    • The keys for all page and frame templates in the book are Key<ResourceRefList>s (and do not use sets), so we should be careful not to add repeating elements.
    • Add a simple check in TemplateUtil.createTemplate(String, ImmList<TemplatedKey<?>>, ElementView, ResourceRefR4, boolean, boolean)
  • The initial element from which a template has been created is not updated when the template changes:
    • The reason for this is that the element does not use the template.
    • Applying a template to the element from which it was created will cause too many problems and will not be implemented (the experience team are ok with this).
      • The major problem is that if we have a page with pink background and save it as template 'Template A', change its border to purple and try to update 'Template A', we will lose the background.

Changed 10 years ago by deni

comment:5 Changed 10 years ago by meddle

  • Status changed from s3b_implementation_finished to s3c_implementation_ok
  • Design_score changed from 0 to 4
  • Imp._score changed from 0 to 4

Merged at [8840].

comment:6 Changed 9 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.