Edgewall Software
Modify

Opened 15 years ago

Closed 15 years ago

#333 closed defect (fixed)

Target platform option got deleted

Reported by: manu.blot@… Owned by: osimons
Priority: minor Milestone: 0.6
Component: Administration interface Version: dev
Keywords: Cc:
Operating System: BSD

Description

from branches/experimental/trac-0.11

From the Builds/Configurations? admin panel:

  1. Select "Add platform"
  2. Enter a new rule (property name, match pattern) pair
  3. Click on the + button to add a new rule

-> The Rules group is cleared up, losing the first rule.

Workaround: From the Builds/Configurations? admin panel:

  1. Select "Add platform"
  2. Enter a new rule (property name, match pattern) pair
  3. Click 'Add'
  4. Re-select the "Add platform" button
  5. The platform rules can now be edited / augmented without losing content

Attachments (1)

t333-new_target_platform-r657.diff (7.8 KB) - added by osimons 15 years ago.
Adds a "New platform" fieldset to the right of the platform listing.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 15 years ago by ebray

Yes, I'm aware of this one. It's a problem in the 0.10 version as well. It will take a little refactoring to fix, but it's definitely worth addressing.

comment:2 Changed 15 years ago by osimons

  • Owner changed from cmlenz to osimons

Yup, this UI glitch needs fixing. Problem is that rules cannot be submitted and stored as the platform does not yet exist. My preference would be to follow similar Trac admin web pages, and put a simple "Add Platform" fieldset to the right of the platform listing on the config page. A simple form with just 'name' and 'create' button, and then after creating the user is redirected to the platform edit where the rules can be added.

Any better ideas? I'll prepare a patch for the my idea anyway.

Changed 15 years ago by osimons

Adds a "New platform" fieldset to the right of the platform listing.

comment:3 Changed 15 years ago by osimons

#170 closed as duplicate.

comment:4 Changed 15 years ago by jhampton

Just applied the patch and it works nicely

comment:5 Changed 15 years ago by osimons

  • Resolution set to fixed
  • Status changed from new to closed

I've reworked the UI in [681] based on last patch.

Add Comment

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain osimons.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.