Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Freeze pane should use xsplit/ysplit instead of topLeftCell #260

Closed
1 of 3 tasks
adriencohen opened this issue Oct 23, 2017 · 0 comments
Closed
1 of 3 tasks

Freeze pane should use xsplit/ysplit instead of topLeftCell #260

adriencohen opened this issue Oct 23, 2017 · 0 comments

Comments

@adriencohen
Copy link

This is:

What is the expected behavior?

The freezed pane should be the same after loading and saving an xlsx file

What is the current behavior?

The freezed pane changes of place.

For example if I have a xlsx with this
<pane xSplit="5" ySplit="11" topLeftCell="GG646" activePane="bottomRight" state="frozen"/>
It displays this in Excel and LibreOffice
xlsx

topLeftCell defines the bottom right pane to show but not the "frozen" part, we need to use xsplit/ysplit to define the frozen part. The current version uses topLeftCell, so with topLeftCell="GG646" you will not be able to navigate in your spreassheet because most rows/columns are frozen.

What are the steps to reproduce?

I don't think that this bug can't be reproduce without an example file.
By executing this script you will see that the frozen pane (top left one) will move.
strange.xlsx

<?php


$reader = new \PhpOffice\PhpSpreadsheet\Reader\Xlsx();
$spreadsheet = $reader->load('strange.xlsx');
$writer = new \PhpOffice\PhpSpreadsheet\Reader\Xlsx($spreadsheet);
$writer->save("out.xls");

Which versions of PhpSpreadsheet and PHP are affected?

PhpSpreadsheet : 1.0.0-beta
php -v
PHP 7.1.2 (cli) (built: Feb 23 2017 19:07:37) ( NTS )
Copyright (c) 1997-2017 The PHP Group
Zend Engine v3.1.0, Copyright (c) 1998-2017 Zend Technologies

Dfred pushed a commit to Dfred/PhpSpreadsheet that referenced this issue Nov 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

1 participant