Edgewall Software
Modify

Opened 8 months ago

Closed 2 months ago

#13116 closed enhancement (fixed)

A few stylistic improvements (#12640)

Reported by: Christian Boos Owned by: Christian Boos
Priority: normal Milestone: 1.3.4
Component: general Version: 1.3dev
Severity: normal Keywords: pullrequest css
Cc: Branch: log:cboos.git@t12640-refinement.5 log:cboos.git@t13116-experimental.1
Release Notes:

Improved styling of wiki and ticket pages.

API Changes:

Description (last modified by Christian Boos)

I had these changes locally for almost a year (and a half)…

Finally getting around to commit them, but as it's been a long time, I'd like to ask for a code review.

[OT] what about a new "pull request" ticket type?

Attachments (0)

Change History (11)

comment:1 by Jun Omae, 8 months ago

In comment:27:ticket:633 and comment:29:ticket:633, I've suggested that font-size of root element should be fixed (14px). The fixed font-size is conflicted with changes for unit length in the proposed changes. Please check the changes with the fixed font-size.

Last edited 8 months ago by Jun Omae (previous) (diff)

in reply to:  description ; comment:2 by Ryan J Ollos, 8 months ago

Replying to Christian Boos:

[OT] what about a new "pull request" ticket type?

Or we could use a keyword, since "pull request" and task/defect/enhancement are not mutually exclusive.

comment:3 by Ryan J Ollos, 8 months ago

Proposed changes look good to me.

in reply to:  2 comment:4 by Christian Boos, 8 months ago

Keywords: pullrequest added

Replying to Ryan J Ollos:

Replying to Christian Boos:

[OT] what about a new "pull request" ticket type?

Or we could use a keyword, since "pull request" and task/defect/enhancement are not mutually exclusive.

Well, I suggested a ticket type as I was thinking about type-specific workflow, type-specific fields and type-specific presentation… I ought to know that this is not yet possible ;-) So in the meantime, sure, a keyword would do!

comment:5 by Christian Boos, 8 months ago

Branch: log:cboos.git@t12640-refinement.5
Description: modified (diff)
Owner: set to Christian Boos
Status: newassigned

comment:6 by Christian Boos, 7 months ago

Changesets from the branch committed in [16866:16871].

There are two additional stylistic changes in log:cboos.git@t13116-experimental, you can see them in testing (check #444 there). It's WIP.

in reply to:  6 comment:7 by Christian Boos, 7 months ago

Branch: log:cboos.git@t12640-refinement.5log:cboos.git@t12640-refinement.5 log:cboos.git@t13116-experimental

Replying to Christian Boos:

There are two additional stylistic changes … you can see them in testing (check #444 there)

So I think I reached what I wanted there. Opinions?

comment:8 by Christian Boos, 7 months ago

Branch: log:cboos.git@t12640-refinement.5 log:cboos.git@t13116-experimentallog:cboos.git@t12640-refinement.5 log:cboos.git@t13116-experimental.1

comment:9 by Ryan J Ollos, 5 months ago

These changes look good to me.

comment:10 by Ryan J Ollos, 2 months ago

If there are no objections, I will go ahead and push these changes.

comment:11 by Ryan J Ollos, 2 months ago

Release Notes: modified (diff)
Resolution: fixed
Status: assignedclosed

Committed to trunk in r16918.

Modify Ticket

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

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.