|
|
domruf
|
1fa33126ed25
|
8 years ago
|
|
less: make pull request 'next iteration' changeset table smaller and scrollable This property was lost in 88ce09daea37. When the whole list is made scrollable, the graph can no longer be positioned absolute. Instead, make it relative so it scrolls with the content ... and 0 height (with the graph overflowing) so the table content will be shown next to the overflowing graph.
|
|
|
domruf
|
7603ce3383c4
|
8 years ago
|
|
|
|
|
domruf
|
9b7eebc852d7
|
8 years ago
|
|
less: disable blue outline in navbar when tabbing through the entries in the top navbar in Chrome
Bootstrap has
a:focus { outline: 5px auto -webkit-focus-ring-color; }
which (inconsistently) puts a blue outline that doesn't match the color scheme.
For now, just disable it in the navbar.
|
|
|
domruf
|
188450ba880e
|
8 years ago
|
|
less: remove horizontal separator lines between settings groups
The separator lines are not necessary - the hanging indentation label should be enough.
|
|
|
domruf
|
13b202c88e41
|
8 years ago
|
|
less: only override Bootstrap default styling of pre where it really is needed
Bootstrap changes the styling of pre, even without special classes.
Thus, override the bootstrap default styling in a less broad manner.
The only thing that really needs to be changed is the border, margin, padding and the background for the code diffs.
|
|
|
Mads Kiilerich
|
eaf600434cf5
|
8 years ago
|
|
less: change markup and styling for the logo shown in the top left corner
Based on work by Dominik Ruf, but in this changeset, the goal is just to improve markup and styling while keeping exactly the same look and feel. In that process, it also fixes some browser specific issues that gave odd spacing/positioning.
All styling (especially the logo image URL) is moved to CSS/LESS, using background-image styling instead of an img tag.
The logo image is made a :before on the site branding text and will usually have to align with this text, so we use inline-block to align to the baseline.
We set the right amount of negative margin-bottom so the image can go below the baseline.
We use a "big enough" negative margin-top to allow the image to bleed outside the line height and into the 12px top padding.
We assume the background image doesn't need cropping or panning and do thus not set background-size, background-position, or background-repeat.
|
|
|
domruf
|
079b24620dce
|
8 years ago
|
|
less: move non-generated parts of pygments.css to less file
The remaining parts of pygments.css can be generated with:
pygmentize -S default -f html | sed 's/^\(.*\) { /.code-highlight \1, .codehilite \1 { /g' > kallithea/public/css/pygments.css
|
|
|
domruf
|
f470ea0a35d2
|
8 years ago
|
|
|
|
|
domruf
|
22dfe2da4fc3
|
8 years ago
|
|
|
|
|
domruf
|
eee7674cef41
|
8 years ago
|
|
|
|
|
domruf
|
5a5398a5bf83
|
8 years ago
|
|
|
|
|
domruf
|
692484bbd445
|
8 years ago
|
|
less: move non contextbar specific style to kallithea.less
Also this is only relevant for icon-empty and icon-gravatar.
|
|
|
domruf
|
9136c3c1680e
|
8 years ago
|
|
less: use bootstrap default font
I think it looks much better.
|
|
|
domruf
|
b983522e7c13
|
8 years ago
|
|
less: use same color for spaces characters
The light ones were barely visible and the others looked almost like normal characters. Also be more explicit and use border-color instead of color.
|
|
|
domruf
|
1cd6e633bab8
|
8 years ago
|
|
less: simplify positioning of the +/- symbols on diff lines
As before, we place the +/- symbols in the pre:before so they default to the same font and styling as the line content.
Instead of giving the pre a wide left margin and put the :before out there using negative positing and visible overflow, just give the :before a fixed width on all lines.
This change has been modified by Mads Kiilerich.
|
|
|
domruf
|
ecd53d8e4adc
|
8 years ago
|
|
|
|
|
domruf
|
a73ba03097fa
|
8 years ago
|
|
|
|
|
domruf
|
6bef1d7bafa6
|
8 years ago
|
|
|
|
|
domruf
|
0f8ac13da245
|
8 years ago
|
|
less: clean up pull request style
- remove unnecessary float: left for .reviewer_status - remove unnecessary height and padding for .reviewers_member - there is no .email_entry, .ip_entry, .pullrequests_section_head, .pr-desc - remove unnecessary margin for .msg-div
|
|
|
domruf
|
d55455f5aefd
|
8 years ago
|
|
less: remove dead styling of #content content
- there is no table .selected - there is no .ui-selectmenu or .ui-selectmenu-focus
|
|
|
domruf
|
e9ba69d3b367
|
8 years ago
|
|
less: add comment for statistics charts and simplify selector
In case we decide to change the element type, make the selector more generic.
|
|
|
domruf
|
7d200066cc48
|
8 years ago
|
|
less: clean up statistics style
- #legend_data default padding is fine - #legend_container border: none is unnecessary - #legend_choices is invisible - #legend_container width: auto is unnecessary - with many commiters less padding and height is actually better - there is no .legend ul
|
|
|
domruf
|
e70bacdb1de7
|
8 years ago
|
|
|
|
|
domruf
|
311e881d1a56
|
8 years ago
|
|
less: clean up .code-browser style
- remove unnecessary style - use default table heading style - move icon spacing style to separate block
|
|
|
domruf
|
4c5073955784
|
8 years ago
|
|
less: textareas should be at least 100px high and 400px wide
Smaller textareas look odd.
|
|
|
domruf
|
900c99095c62
|
8 years ago
|
|
|
|
|
domruf
|
6351f10ed2bc
|
8 years ago
|
|
|
|
|
domruf
|
065690dba16b
|
8 years ago
|
|
less: drop special summary style
- remove special margin - use the same style for all descriptions (same as .formatted-fixed) - there is no #content div.panel #summary p
|
|
|
domruf
|
e6c0425f5833
|
8 years ago
|
|
|
|
|
domruf
|
7e149192a0e9
|
8 years ago
|
|
|
|
|
domruf
|
26c20ab844cc
|
8 years ago
|
|
|
|
|
domruf
|
ed783f8d4562
|
8 years ago
|
|
|
|
|
domruf
|
7f3486d1e084
|
8 years ago
|
|
less: remove .readme style
I think the new bootstrap default actually looks better.
|
|
|
domruf
|
5ec18fe629a5
|
8 years ago
|
|
|
|
|
domruf
|
fd22ce18d312
|
8 years ago
|
|
|
|
|
domruf
|
adf1dd34f3a0
|
8 years ago
|
|
|
|
|
domruf
|
1ea5d93c97de
|
8 years ago
|
|
less: remove unnecessary repo size styling
- empty badges are already hidden - the badge style overrides the padding anyway
|
|
|
domruf
|
80fffda17c55
|
8 years ago
|
|
|
|
|
domruf
|
a654ce23d552
|
8 years ago
|
|
less: simplify .expand_commit selector
also - remove unnecessary width style - move to separate block and add comment
|
|
|
domruf
|
ddba357cb1e3
|
8 years ago
|
|
|
|
|
domruf
|
c52f18a1c2c7
|
8 years ago
|
|
less: clean up lang_stats style
- remove background-color bootstrap already uses @brand-primary - text-decoration: none is unnecessary - move to separate block and add comment
|
|
|
domruf
|
f56a7b14b221
|
8 years ago
|
|
less: use default datatable header (without) background color
This has a better, cleaner look.
|
|
|
domruf
|
89be46d1bfd0
|
8 years ago
|
|
less: remove .reviewer_ac
It has nothing to do with reviewers or autocomplete and the 'padding: 10px' does not improve the style.
|
|
|
domruf
|
0b9db9c27399
|
8 years ago
|
|
|
|
|
domruf
|
75ad179350b3
|
8 years ago
|
|
less: remove unnecessary table pre style
difftable code stuff is already set in kallithea-diff.less
|
|
|
domruf
|
39fc6313349a
|
8 years ago
|
|
less: remove unused table column styling
I could not find a place where these are used.
|
|
|
domruf
|
6291fafda89a
|
8 years ago
|
|
less: change diff-actions coloring
<a> already use @brand-primary color and I think it is actually beneficial if the two other non-link elements look a bit different. (blackish default .panel-heading color)
The two non-link elements thus already have a different non-link look and don't need the opacity.
|
|
|
domruf
|
ccf1ccc77ca3
|
8 years ago
|
|
less: don't override default label font-weight
Bootstrap makes it bold by default. Use that so we don't have to specify bold when we need it.
The places where we don't want bold, the styling must be overridden.
|
|
|
domruf
|
d1609414c3e8
|
8 years ago
|
|
less: remove dead traffic styling
There is no .traffic class. It was apparently once a name for the repo statistics.
|
|
|
domruf
|
22a23f38e472
|
8 years ago
|
|
less: remove vertical-align: inherit for .radio-inline and .checkbox-inline
The effect is minimal and I'd argue it looks even a little bit better without it.
|
|
|
domruf
|
647e7e0ce812
|
8 years ago
|
|
|
|
|
domruf
|
42932bee7a37
|
8 years ago
|
|
|
|
|
domruf
|
6f0b598499d8
|
8 years ago
|
|
|
|
|
domruf
|
4f183cfd4ceb
|
8 years ago
|
|
|
|
|
domruf
|
c0f455bad77b
|
8 years ago
|
|
less: stick with the default of no shadow in navbar
And .icon is a span now, so display: inline has no effect
|
|
|
domruf
|
334870b15a82
|
8 years ago
|
|
|
|
|
domruf
|
d69d258c1403
|
8 years ago
|
|
less: use more generic selector for formatted-fixed
Why should this only be usable on <div>!
Also add comment.
|
|
|
domruf
|
7c0f072dfc1e
|
8 years ago
|
|
less: remove weird .anchor style
I see no point in this.
|
|
|
domruf
|
7c1112c0b2de
|
8 years ago
|
|
less: simplify permalink style
- 'position: absolute' and 'margin: 3px 4px;' make it actually look worse - 'text-decoration: none' is already set on all <a> - we only use it in one place so the selector can be simplified
Also add comment.
|
|
|
domruf
|
00703cdbd343
|
8 years ago
|
|
|
|
|
domruf
|
7b7b59416910
|
8 years ago
|
|
less: remove line-through for <del>
- line-through makes it hard to read - since the text is already red it is kinda redundant - it gets overriden by kallithea-diff.less line 87 anyway
|
|
|
domruf
|
01150c6eedc0
|
8 years ago
|
|
less: remove unnecessary diff table style in kallithea-diff.less
I see no place where these are necessary - probably because Bootstrap styling.
|
|
|
domruf
|
a92a06789f56
|
8 years ago
|
|
less: remove unnecessary width 100% for diff code
table.code-difftable already has width: 100%
|
|
|
domruf
|
ceaa65df5add
|
8 years ago
|
|
diff: use fontello icon-diff-* names in generated html
This way we need less style code ... but generate slightly bigger html.
Also move the style code to kallithea-diff.less and remove the now obsolete comment.
|
|
|
domruf
|
b66e4d720198
|
8 years ago
|
|
templates: change .node elements to <span>
This way we don't need to make them 'float: left'
|
|
|
domruf
|
6db2e693ee1e
|
8 years ago
|
|
less: remove unused .code-difftable .code td styling
Every place I found .code was set to th <td> so this is dead code.
|
|
|
domruf
|
ed5f5dd98823
|
8 years ago
|
|
less: clean up unnecessary diff line-no styling
- less spacing still leave room to show the comment bubble with hiding the line number - looks good w/o letter-spacing - bootstrap already sets box-sizing: border-box; - it is a <a> no need for cursor: pointer - width: 100% makes no difference
|
|
|
domruf
|
b297ba29d93e
|
8 years ago
|
|
|
|
|
domruf
|
63ebe4815723
|
8 years ago
|
|
|
|
|
domruf
|
7f45efdd3f12
|
8 years ago
|
|
less: remove unnecessary .cur_cs
The class is only used once on a h6, which already is bold and has top and bottom 10px margin. The 2px margin left and right are not really helpful.
|
|
|
domruf
|
4d339efccf64
|
8 years ago
|
|
less: tag style should always apply not only if placed inside .right element
Besides, .right is no longer used.
|
|
|
domruf
|
bee750d4ffff
|
8 years ago
|
|
|
|
|
domruf
|
9b4ebc0b2742
|
8 years ago
|
|
less: add hint about how to generate css from less files
In case somebody opens the css file to make some changes, he will see this message on how to use less instead of editing the css files.
|
|
|
Mads Kiilerich
|
d44232c8d76a
|
8 years ago
|
|
less: fix displaying +/- signs in code-difftable 35d3a85fc650 introduced the Bootstrap default overflow:auto styling of pre. That made overflow hidden instead of visible, and thus broke our display of +/- as a :before with relative positioning outside the pre. To undo that, add overflow to the list of Bootstrap pre styling we override. Based on another fix by Dominik Ruf.
|
|
|
Thomas De Schampheleire
|
ebb6f89b8d22
|
8 years ago
|
|
|
|
|
Thomas De Schampheleire
|
386501cd9e00
|
8 years ago
|
|
|
|
|
Thomas De Schampheleire
|
3f1e3fba3a3e
|
8 years ago
|
|
style: move perm_tag to kallithea-tags.less
Also move these styles in the original style.css to make the delta again zero.
|
|
|
Thomas De Schampheleire
|
354adbe687a3
|
8 years ago
|
|
style: less'en select2 styles
Some steps towards the version of kallithea-select2.less prepared by Dominik Ruf.
|
|
|
Thomas De Schampheleire
|
cf0e5b32c063
|
8 years ago
|
|
style: change indentation of stylesheet to match future less output
Align the indentation of the stylesheet with the indentation that a less-compiled version of this stylesheet would use. This will allow easy comparison of the original and less-generated stylesheet in future commits.
|
|
|
Thomas De Schampheleire
|
b9896fba8b05
|
8 years ago
|
|
style: remove empty lines in stylesheet to match future less output
Align the formatting of the stylesheet with the formatting that a less-compiled version of this stylesheet would use. This will allow easy comparison of the original and less-generated stylesheet in future commits.
|
|
|
Thomas De Schampheleire
|
89e1117dbfbb
|
8 years ago
|
|
style: fix formatting oddities and bugs
This will also align the formatting of the stylesheet with the indentation that a less-compiled version of this stylesheet would use. This will allow easier comparison of the original and less-generated stylesheet in future commits.
|
|
|
domruf
|
7e22d42320cd
|
8 years ago
|
|
style: be consistent and don't use col-*-* classes
In order to not show overlap the lock icon also change its positioning style.
|
|
|
Mads Kiilerich
|
1ba4fee6b27c
|
8 years ago
|
|
|
|
|
domruf
|
2ce1f4cd87d8
|
8 years ago
|
|
navbar: use Bootstrap 'caret' class instead of our own 'navbar-caret' This mostly reverses e075f2cc4f8c - now we use so much Bootstrap that it no longer is an issue. Note: This icon is smaller than the old one.
|
|
|
domruf
|
789de118156b
|
8 years ago
|
|
style: consistently use 'changeset_hash' class for monospace hashes
Get rid of 'revision-link' and some use of 'hash'.
There is still some use of 'hash' for columns, but the content there already has 'changeset_hash'.
|
|
|
Mads Kiilerich
|
4f3447149c98
|
8 years ago
|
|
changelog: update styling - don't restrict styling to graph when it is all for all changelog views
Be more specific when styling applies to all table#changesets or to columns in it.
|
|
|
domruf
|
90282e663313
|
8 years ago
|
|
changelog: avoid cutting off commit messages
Split commit message into 2 <div> elements: one with just the first line, and one with the full message. This makes it easier to handle the toggling of showing the whole commit message. This way, we no longer have to mess with the line height, and we avoid problems with cutting off parts of the text below the text baseline.
Also stop cutting off the first line if it is too long for the table column.
It adds a bit of overhead to store the first line twice, but having them separate make things simpler and is worth it.
Patch modified by Mads Kiilerich.
|
|
|
Mads Kiilerich
|
fa55523f3790
|
8 years ago
|
|
changelog: fix column sizing - don't expand the date column but give everything to the description column
The date column inherited 33% width - for now, just work around that.
|
|
|
Mads Kiilerich
|
f77ef41b39aa
|
8 years ago
|
|
changelog: dim the whole row for changesets that are merges or out of range Backout b99cd2bc7540 - we can do that after 2f35bd7b97aa moved the tooltips so they don't inherit opacity any more.
|
|
|
domruf
|
19f40a5088d4
|
8 years ago
|
|
templates: don't abuse elements as links or buttons
Bootstrap recommends using <button>, so most of the time we should use that.
Except if we don't want the additional padding of a button (like for the expand 'button' on the changelog page).
|
|
|
domruf
|
387b8d551003
|
8 years ago
|
|
style: rely on css to truncate author name in tables instead of truncating to fixed length
Dynamic truncation allows us in the future to use different length on different screen sizes.
|
|
|
Mads Kiilerich
|
4daff1d77dd4
|
8 years ago
|
|
statistics: make statistics view work again 88ce09daea37 removed sizing of the graph, causing Flot javascript to fail without showing anything.
|
|
|
Andrew Shadura
|
2e72d2d16a0f
|
8 years ago
|
|
comments: display comment previews while submitting
Instead of just saying 'Submitting' and not showing any progress to the user until the comment has been accepted by the server, show a preview of the comment above the comment box in a way which is makes it obvious to the user the comment is being submitted. Apart from that, also clear the comment box so that a repeated clicking the submit button doesn't result in a duplicate comment.
The preview doesn't highlight URLs or support @mentions or *bold*, which is a good enough approximation in this case. When/if we (re-)add the rST/Markdown support, we will need a client-side parser for the syntax we choose.
When the submission fails, display a message and offer the user to retry or cancel the submission.
|
|
|
Mads Kiilerich
|
91182a358fb3
|
8 years ago
|
|
templates: drop odd grid_edit and grid_delete classes
We really should have both buttons inside the same float-left so they get proper spacing ... but that doesn't work when one is wrapped in a form ...
|
|
|
Mads Kiilerich
|
e4d2fec64955
|
8 years ago
|
|
autocompletion: drop explicit container elements - just create them when necessary
The previous wide styling of the @mention selection list didn't look pretty - just use the default width.
Inspired by select2 port by Dominik Ruf.
|
|
|
domruf
|
8b265e5f9e07
|
8 years ago
|
|
|
|
|
domruf
|
9af9a8966d6d
|
8 years ago
|
|
changeset: use bootstrap label instead of custom css
Also replace old .right with .pull-right for correct floating.
|
|
|
domruf
|
3a65e48d81fe
|
8 years ago
|
|
templates: make gist edit and new page look more similar
- remove gravatar from edit page - add no-padding to edit page - use the same order and alignment of the mimetype and filename elements on both pages
|
|
|
domruf
|
b89511f1aa24
|
8 years ago
|
|
template: simplify journal styling
This requires less css code.
|
|
|
Mads Kiilerich
|
ffb70a338ce1
|
8 years ago
|
|
style: fix display of the PR graph of changesets available for update 88ce09daea37 dropped explicit html styling - fix it by adding css styling.
|