Re: Side-by-side diff broken in some browsers

2018-02-14 Thread Mads Kiilerich

On 02/14/2018 03:23 PM, Thomas De Schampheleire wrote:

What is the status of this? I see the duplication removed on the
default branch, but the actual issue still persists.


I pushed a fix. Thanks for reporting and testing.

/Mads
___
kallithea-general mailing list
kallithea-general@sfconservancy.org
https://lists.sfconservancy.org/mailman/listinfo/kallithea-general


Re: Side-by-side diff broken in some browsers

2018-02-14 Thread Mads Kiilerich

On 02/14/2018 03:23 PM, Thomas De Schampheleire wrote:

Hi Dominik,

2018-01-29 19:38 GMT+01:00 Dominik Ruf :

Hi Thomas,

thanks for the hint.
There are 2 problems.
1. there should of course be only one style entry for this
2. the 2 way diff should not be put in another panel

I'll push 2 more changesets to fix this.

What is the status of this? I see the duplication removed on the
default branch, but the actual issue still persists.
Is it supposed to be fixed, or is the commit that should fix it not yet applied?


I think it is at
https://kallithea-scm.org/repos/kallithea-incoming/changeset/d6fb685267846cab2b4b2f3655363bb63551ad52#comment-1861

/Mads
___
kallithea-general mailing list
kallithea-general@sfconservancy.org
https://lists.sfconservancy.org/mailman/listinfo/kallithea-general


Re: Side-by-side diff broken in some browsers

2018-02-14 Thread Thomas De Schampheleire
Hi Dominik,

2018-01-29 19:38 GMT+01:00 Dominik Ruf :
> Hi Thomas,
>
> thanks for the hint.
> There are 2 problems.
> 1. there should of course be only one style entry for this
> 2. the 2 way diff should not be put in another panel
>
> I'll push 2 more changesets to fix this.

What is the status of this? I see the duplication removed on the
default branch, but the actual issue still persists.
Is it supposed to be fixed, or is the commit that should fix it not yet applied?

Thanks,
Thomas
___
kallithea-general mailing list
kallithea-general@sfconservancy.org
https://lists.sfconservancy.org/mailman/listinfo/kallithea-general


Re: Side-by-side diff broken in some browsers

2018-01-29 Thread Dominik Ruf
Hi Thomas,

thanks for the hint.
There are 2 problems.
1. there should of course be only one style entry for this
2. the 2 way diff should not be put in another panel

I'll push 2 more changesets to fix this.

cheers
Dominik

Thomas De Schampheleire  schrieb am Mo., 29.
Jan. 2018 um 13:27 Uhr:

> Hi Dominik,
>
> After upgrading our Kallithea to something based on almost latest
> default branch, we noticed an issue with side-by-side diff in some
> browsers, e.g.
> - Firefox 52 (Windows)
> - Chrome 63 (Windows)
>
> I don't see the issue on:
> - Firefox 57 (Linux)
>
> The issue is that the right pane does not seem to contain any code,
> and as such there is no diff.
> Please see screenshots. Second screenshot is same page as first but
> scrolled down a bit.
>
> I bisected the issue down to the commit below.
> I confirmed that by removing that block on the latest default, two-way
> diff works fine again. However, I don't know what else I broke now.
>
> Best regards,
> Thomas
>
>
> # HG changeset patch
> # User domruf 
> # Date 1509208744 -7200
> #  Sat Oct 28 18:39:04 2017 +0200
> # Node ID eee7674cef4126d3ad15754992ac1eb20f4587d8
> # Parent  5a5398a5bf831d33808e0d6261a0516b27366cd2
> less: prevent elements from overflowing panels
>
> diff --git a/kallithea/public/css/style.css
> b/kallithea/public/css/style.css
> --- a/kallithea/public/css/style.css
> +++ b/kallithea/public/css/style.css
> @@ -102,6 +102,12 @@ nav.navbar #logo > .navbar-brand > img {
>margin-top: -9px;
>margin-right: 5px;
>  }
> +/* every direct child of a panel, that is not .panel-heading, should auto
> + * overflow to prevent overflowing of elements like text boxes and tables
> */
> +.panel > :not(.panel-heading) {
> +  overflow-x: auto;
> +  min-height: 0.01%;
> +}
>  /* language bars (summary page) */
>  #lang_stats .progress-bar {
>min-width: 15px;
> diff --git a/kallithea/public/less/style.less
> b/kallithea/public/less/style.less
> --- a/kallithea/public/less/style.less
> +++ b/kallithea/public/less/style.less
> @@ -111,6 +111,13 @@ nav.navbar #logo > .navbar-brand > img {
>margin-right: 5px;
>  }
>
> +/* every direct child of a panel, that is not .panel-heading, should auto
> + * overflow to prevent overflowing of elements like text boxes and tables
> */
> +.panel > :not(.panel-heading){
> +  overflow-x: auto;
> +  min-height: 0.01%;
> +}
> +
>  /* language bars (summary page) */
>  #lang_stats {
>.progress-bar {
>
___
kallithea-general mailing list
kallithea-general@sfconservancy.org
https://lists.sfconservancy.org/mailman/listinfo/kallithea-general