Skip to content
This repository has been archived by the owner on Aug 9, 2024. It is now read-only.

Field note visual height glitch on Chrome #1428

Closed
fabriciomurta opened this issue Feb 3, 2017 · 3 comments
Closed

Field note visual height glitch on Chrome #1428

fabriciomurta opened this issue Feb 3, 2017 · 3 comments
Assignees
@fabriciomurta
Labels
3.x 4.x defect

Comments

@fabriciomurta
Copy link
Contributor

fabriciomurta commented Feb 3, 2017 •
edited
Loading

Found: 4.2.0
Ext.NET forum thread: Bug when displaying Field 3.x and 4.x

On chrome, the spacing between fields looks broken, while it works fine in IE and FireFox.

This simple CSS override seems to address the issue:

<style type="text/css">
    .x-form-item-body {
        height: initial
    }
</style>
@fabriciomurta fabriciomurta added this to the 4.2.1 milestone Feb 3, 2017
@fabriciomurta
Copy link
Contributor Author

Notice, this issue is not related to #1417 as it happens also on Ext.NET 3.x.

@fabriciomurta fabriciomurta mentioned this issue Feb 10, 2017
Open
fabriciomurta added a commit to extnet/examples5.ext.net that referenced this issue Feb 10, 2017
@fabriciomurta
Fixes display issues with the FieldNote Overview example.
31db645 
One of ths issues were introduced by chrome 56 and is reported under
extnet/Ext.NET#1428. This should be reverted as soon as the issue (or
chrome) is fixed.

It also adds mechanism to make the form panel fit its contents depending
on the theme.

Related github issue: #12.

Issue extnet/Ext.NET#1431 tracks removing this override on next Ext.NET
public update.
@fabriciomurta fabriciomurta mentioned this issue Feb 23, 2017
Open
fabriciomurta added a commit to extnet/examples5.ext.net that referenced this issue Feb 23, 2017
@fabriciomurta
Adds workaround on Input Mask example to work with Chrome ~v56.
04d270a 
The example now has a CSS override that makes the fields look right with
the field notes.

Related github issues: #13 and extnet/Ext.NET#1428.
@fabriciomurta fabriciomurta modified the milestones: 4.2.1, 4.3.0 May 17, 2017
@geoffreymcgill geoffreymcgill modified the milestones: 4.3.0, 4.3.1, 4.4.0, 4.5.0 Aug 2, 2017
@fabriciomurta
Copy link
Contributor Author

This use an specific Ext.NET feature, so there's no point in trying to reproduce in standalone ExtJS.

@geoffreymcgill geoffreymcgill modified the milestones: 4.5.0, 4.4.1 Sep 20, 2017
@fabriciomurta fabriciomurta self-assigned this Sep 25, 2017
@fabriciomurta
Copy link
Contributor Author

Removed the workaround in the example and the bug no longer happens. This was fixed in either ExtJS 6.5.0 or 6.5.1 releases.

@geoffreymcgill geoffreymcgill removed this from the 4.4.1 milestone Sep 26, 2017
fabriciomurta added a commit to extnet/examples5.ext.net that referenced this issue Sep 27, 2017
@fabriciomurta
Removes field note workaround from examples.
d44ce3b 
The two examples using the field note height workaround had been
reverted as the bug is no longer actual as of ExtJS 6.5.0.

Related github issue: extnet/Ext.NET#1428.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@fabriciomurta fabriciomurta

Labels
3.x 4.x defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@geoffreymcgill @fabriciomurta