Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-06 14:10:11 +01:00
Code Issues Projects Releases Packages Wiki Activity

Apply border-radius when painting borders

Browse source
This commit is contained in:
Simon Sapin 2020-01-03 14:56:59 +01:00
parent 7281336116
commit 962c85b553
3 changed files with 27 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

29
components/layout_2020/display_list.rs
View file

@ -5,11 +5,11 @@
use crate::fragments::{BoxFragment, Fragment};
use crate::geom::physical::{Rect, Vec2};
use embedder_traits::Cursor;
use euclid::{Point2D, SideOffsets2D};
use euclid::{Point2D, SideOffsets2D, Size2D};
use gfx::text::glyph::GlyphStore;
use std::sync::Arc;
use style::properties::ComputedValues;
use style::values::computed::{BorderStyle, Length};
use style::values::computed::{BorderStyle, Length, LengthPercentage};
use webrender_api::{self as wr, units, CommonItemProperties, PrimitiveFlags};
pub struct DisplayListBuilder {
@ -131,9 +131,10 @@ impl BoxFragment {
// TODO(gw): Make use of the WR backface visibility functionality.
flags: PrimitiveFlags::default(),
};
let border_radius = self.border_radius(&border_rect);
self.background_display_items(builder, &common);
self.border_display_items(builder, &common, border_rect);
self.border_display_items(builder, &common, border_rect, border_radius);
let content_rect = self
.content_rect
.to_physical(self.style.writing_mode, containing_block)
@ -143,6 +144,25 @@ impl BoxFragment {
}
}
fn border_radius(&self, border_rect: &units::LayoutRect) -> wr::BorderRadius {
let resolve = |radius: &LengthPercentage, box_size: f32| {
radius.percentage_relative_to(Length::new(box_size)).px()
};
let corner = |corner: &style::values::computed::BorderCornerRadius| {
Size2D::new(
resolve(&corner.0.width.0, border_rect.size.width),
resolve(&corner.0.height.0, border_rect.size.height),
)
};
let b = self.style.get_border();
wr::BorderRadius {
top_left: corner(&b.border_top_left_radius),
top_right: corner(&b.border_top_right_radius),
bottom_right: corner(&b.border_bottom_right_radius),
bottom_left: corner(&b.border_bottom_left_radius),
}
}
fn background_display_items(
&self,
builder: &mut DisplayListBuilder,
@ -161,6 +181,7 @@ impl BoxFragment {
builder: &mut DisplayListBuilder,
common: &CommonItemProperties,
border_rect: units::LayoutRect,
radius: wr::BorderRadius,
) {
let b = self.style.get_border();
let widths = SideOffsets2D::new(
@ -192,7 +213,7 @@ impl BoxFragment {
right: side(b.border_right_style, b.border_right_color),
bottom: side(b.border_bottom_style, b.border_bottom_color),
left: side(b.border_left_style, b.border_left_color),
radius: wr::BorderRadius::zero(),
radius,
do_aa: true,
});
builder.wr.push_border(common, border_rect, widths, details)

2
components/style/properties/longhands/border.mako.rs
View file

@ -75,7 +75,7 @@
"BorderCornerRadius",
"computed::BorderCornerRadius::zero()",
"parse",
engines="gecko servo-2013",
engines="gecko servo-2013 servo-2020",
extra_prefixes=prefixes,
spec=maybe_logical_spec(corner, "radius"),
boxed=True,

2
components/style/properties/shorthands/border.mako.rs
View file

@ -239,7 +239,7 @@ pub fn parse_border<'i, 't>(
<%helpers:shorthand
name="border-radius"
engines="gecko servo-2013"
engines="gecko servo-2013 servo-2020"
sub_properties="${' '.join(
'border-%s-radius' % (corner)
for corner in ['top-left', 'top-right', 'bottom-right', 'bottom-left']