Edgewall Software
Modify

Opened 5 years ago

Closed 5 years 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:
Internal 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, 5 years ago

In comment:27:ticket:633 and comment:29:ticket:633, I suggest 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.

Version 0, edited 5 years ago by Jun Omae (next)

in reply to:  description ; comment:2 by Ryan J Ollos, 5 years 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, 5 years ago

Proposed changes look good to me.

in reply to:  2 comment:4 by Christian Boos, 5 years 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, 5 years ago

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

comment:6 by Christian Boos, 5 years 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, 5 years 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, 5 years 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 years ago

These changes look good to me.

comment:10 by Ryan J Ollos, 5 years ago

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

comment:11 by Ryan J Ollos, 5 years 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'.
to The owner will be changed from Christian Boos to the specified user.

Add Comment


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