Titanium JIRA Archive
Titanium SDK/CLI (TIMOB)

[TIMOB-10166] Android: BorderRadius on TableView doesn't clip content on Android, Ti 2.1.0

GitHub Issuen/a
TypeBug
PriorityHigh
StatusClosed
ResolutionFixed
Resolution Date2012-09-19T17:01:13.000+0000
Affected Version/sRelease 2.1.0
Fix Version/sRelease 2.1.3, Release 3.0.0, Sprint 2012-19 Core, 2012 Sprint 19
ComponentsAndroid
LabelsSupportTeam, community, core, module_tableview, parity, qe-testadded
ReporterQuang Pham
AssigneeAllen Yeung
Created2012-07-13T03:11:21.000+0000
Updated2013-09-26T11:03:04.000+0000

Description

Steps to Reproduce

Titanium.UI.setBackgroundColor('#000'); var win = Titanium.UI.createWindow({ title:'Tab 1', backgroundColor:'#fff' }); var rows = []; for (var i=0; i < 5; i++) { var row = Ti.UI.createTableViewRow({}); row.add(Ti.UI.createView({ width: 100, height: 50, left: 0, backgroundColor: 'blue', borderWidth: 1, borderColor: '#000' })); rows.push(row); }; var table = Ti.UI.createTableView({ data: rows, borderWidth: 1, borderRadius: 10, borderColor: 'blue', backgroundColor: '#fff', rowHeight: 50, top: 100, left: 100, right: 100, height: Ti.UI.SIZE }); win.add(table); win.open();

Actual Result

overflow content is not clipping, override borderRadius

Expected Result

overflow content is clipping

Attachments

FileDateSize
.log2012-07-13T03:12:04.000+00008365021
diagnostic8601487086261290751.log2012-07-13T03:12:15.000+00003825

Comments

  1. Juan G. Hurtado 2012-08-06

    Same issue here. Android 2.1 and 2.2, with Titanium SDK 2.1.0 and 2.1.1
  2. Max Stepanov 2012-08-20

    Possible workaround is to embed table view into generic view with border radius set.
  3. Blain Hamon 2012-08-27

    Note to self: Must investigate feasibility of doing such and get back to Alan by the 28th.
  4. Blain Hamon 2012-08-27

    Talked with other Allen. He implemented something for views, and it wasn't inherited by TableView, but he sees no reason why it couldn't be done. Schedule for next sprint.
  5. Blain Hamon 2012-08-27

    Now that feasibility is known, returning it to Neeraj to be dispatched.
  6. Allen Yeung 2012-09-17

    PR Ready: https://github.com/appcelerator/titanium_mobile/pull/2980
  7. Etienne Champetier 2012-09-19

  8. Allen Yeung 2012-09-19

    Please test this against the most recent PR. It seems to work for me, as in the image shows up.
  9. Allen Yeung 2012-09-19

    Backport for 2_1_X: https://github.com/appcelerator/titanium_mobile/pull/3006
  10. Etienne Champetier 2012-09-20

    my app is working again with ayeung:timob-10166-2_1_X, thanks!
  11. Olga Romero 2012-09-21

    Verified fix with: osx Mountain Lion 10.8.1 Titanium Studio, build: 2.1.2.201208301612 Titanium SDK: 2.1.3.v20120920154617 Titanium SDK: 3.0.0.v20120921144915 Device: Nexus S3 4.0.4

JSON Source