[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-05-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

Roman Eisele b...@eikota.de changed:

   What|Removed |Added

  Component|Libreoffice |Spreadsheet

--- Comment #23 from Roman Eisele b...@eikota.de 2012-05-04 07:28:47 PDT ---
This is a Calc issue, therefore changed 'Component' field accordingly.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-05-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

Rainer Bielefeld libreoff...@bielefeldundbuss.de changed:

   What|Removed |Added

  Component|Spreadsheet |Libreoffice

--- Comment #24 from Rainer Bielefeld libreoff...@bielefeldundbuss.de 
2012-05-04 10:40:10 UTC ---
I still think that this one is a more general Component Libreoffice(or even
Better: UI?) issue. Although I think there are more spreadsheets with many
sheets than DRAWings with lots of layers, behavior should be the same.

BTW, may be for the sake of unifications we should have an add layer + Tab
for DRAW and Impress? May be that can be integrated into the thoughts here?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-04-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #19 from Kohei Yoshida kohei.yosh...@gmail.com 2012-04-27 
08:25:54 PDT ---
(In reply to comment #18)
 Created attachment 60503 [details] [review]
 diff of the fixes, updated to fix the problem Kohei mentioned
 
 Ok, here is a new patch with the problem that Kohei mentioned fixed.
 
 I tried it out by applying it to a slightly older version because the current
 master doesn't build for me (on fedora right now).

Hello Forest,

This patch is definitely in the right direction.  I can see different width of
the visible area depending on the tab count.  I can still see two issues:

1) the width is still not wide enough to show all tabs (including the '+' tab)
when the window is not maximized.  Especially when you set the initial sheet
number to, say, 6, only the first 5 sheet tabs are visible.

2) The patch ignores custom width set by the document.  See Comment 4 for
details.

BTW, this is not a major issue, but it would be great if you could create your
patch by git format-patch, rather than git diff.  See this page

http://wiki.documentfoundation.org/Development/Patch_Handling_Guideline

for general guidelines that we follow.  It's not mandatory, but it would make
our job a bit easier when applying patches.

Thanks a lot!  Sorry it took a little while for me to review.

Kohei

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-04-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #20 from Forest Johnson johns...@beloit.edu 2012-04-27 15:23:53 
PDT ---
(In reply to comment #19)
 1) the width is still not wide enough to show all tabs (including the '+' tab)
 when the window is not maximized.  Especially when you set the initial sheet
 number to, say, 6, only the first 5 sheet tabs are visible.

So, make the window small, initial sheet count to 6, new sheet, and the plus is
hidden?

 2) The patch ignores custom width set by the document.  See Comment 4 for
 details.

The intended functionality is that when you save a document and re-open it the
splitter should be in the same place?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-04-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #21 from Forest Johnson johns...@beloit.edu 2012-04-27 15:25:19 
UTC ---
(In reply to comment #20)

 So, make the window small, initial sheet count to 6, new sheet, and the plus 
 is
 hidden?

excuse me, I meant new spreadsheet document not make a new sheet/tab.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-04-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #22 from Kohei Yoshida kohei.yosh...@gmail.com 2012-04-27 
16:10:06 PDT ---
(In reply to comment #20)
 (In reply to comment #19)
  1) the width is still not wide enough to show all tabs (including the '+' 
  tab)
  when the window is not maximized.  Especially when you set the initial sheet
  number to, say, 6, only the first 5 sheet tabs are visible.
 
 So, make the window small, initial sheet count to 6, new sheet, and the plus 
 is
 hidden?

Make the window smaller but large enough to have enough room to show all the
tabs.  In my case, I just un-maximized the window.

And it's not just the plus tab that's hidden, Sheet6 tab and a part of Sheet5
tab was obscured as well.

  2) The patch ignores custom width set by the document.  See Comment 4 for
  details.
 
 The intended functionality is that when you save a document and re-open it the
 splitter should be in the same place?

Not exactly.  When the user never adjusted the scroll bar width, then you just
set the default width even after saving and re-opening.  But if the user has
adjusted the scroll bar width even just once, that width will be stored with
the document, and the next time the document gets loaded its width should be
restored.

I hope this makes sense...

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-04-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

Forest Johnson johns...@beloit.edu changed:

   What|Removed |Added

  Attachment #58579|0   |1
is obsolete||

--- Comment #18 from Forest Johnson johns...@beloit.edu 2012-04-23 19:32:16 
PDT ---
Created attachment 60503
  -- https://bugs.freedesktop.org/attachment.cgi?id=60503
diff of the fixes, updated to fix the problem Kohei mentioned

Ok, here is a new patch with the problem that Kohei mentioned fixed.

I tried it out by applying it to a slightly older version because the current
master doesn't build for me (on fedora right now).

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-03-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #16 from Kohei Yoshida kohei.yosh...@gmail.com 2012-03-19 
10:01:43 PDT ---
Hi there,

Thanks for working on this.

I've just tested your patch.  But I'm not sure if I see any noticeable
improvement.

What I did is as follows:

1. Open a new spreadsheet document.
2. Go to Tools - Options - LibreOffice Calc - Defaults.
3. Change the number of default worksheets to, say, 5.
4. Click OK.
5. Close the app.
6. Re-open Calc with a new document.
7. the right part of the tab is still obscured.

This results even when the window is large enough to show all tabs (including
the plus tab).

What I would expect is that the default position of the drag handle to be
variable based on the number of sheet tabs such that all sheets be visible when
the window is large enough.

Regards,
Kohei

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-03-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #17 from Kohei Yoshida kohei.yosh...@gmail.com 2012-03-19 
11:23:30 PDT ---
(In reply to comment #15)
 Some other students and myself were wondering about some additional
 enhancements to this bugfix.
 
 Should minimum and maximum tab sizes be enforced (e.g. further truncation of
 sheet names, max character limit, and single character names)?

Are you talking about SC_TABBAR_MIN and SC_SCROLLBAR_MIN in tabview.cxx?  If
so, I don't have a strong opinion about this.  If you feel that it's better to
change them, I won't have any objection.

 Should the boolean (mbTabsLastMovedBySplitter) that is changed when the
 aHSplitter is selected be a right-click menu option that the user can set and
 reset during the same session?

No.

 Should that boolean state also be stored when the document is saved?

Nope.

 Should the active tab widget be shown along with the + widget when the
 splitter is being moved on top of the other tabs? If the tab area is too small
 to show both the active tab and + widgets, the splitter will simply move
 over top of them.

Honestly, I have no idea. ;-)  I would think that showing the active tab should
be the priority.  I wouldn't worry too much about the plus tab.

Hope this helps.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-03-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #15 from Jared McCannon jared.r.mccan...@gmail.com 2012-03-18 
18:32:26 PDT ---
Some other students and myself were wondering about some additional
enhancements to this bugfix.

Should minimum and maximum tab sizes be enforced (e.g. further truncation of
sheet names, max character limit, and single character names)?

Should the boolean (mbTabsLastMovedBySplitter) that is changed when the
aHSplitter is selected be a right-click menu option that the user can set and
reset during the same session?

Should that boolean state also be stored when the document is saved?

Should the active tab widget be shown along with the + widget when the
splitter is being moved on top of the other tabs? If the tab area is too small
to show both the active tab and + widgets, the splitter will simply move
over top of them.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-03-16 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #14 from Forest Johnson johns...@beloit.edu 2012-03-16 15:34:36 
PDT ---
Created attachment 58579
  -- https://bugs.freedesktop.org/attachment.cgi?id=58579
preliminary patch

Hi,

Here is our preliminary patch that somewhat resolves this issue. We decided to
take a different course of action in how we approach the fix.

We made the position of the tab-bar splitter relative to the window size,
rather than pixel-specific. 

We also made the tab bar splitter drag out the tabs as if inside a drawer
instead of leaving empty space.

There could be some issues with this patch. For example, we should probably
apply the same relative behaviour to the phSplitter (the splitter that can be
dragged out from the right side to allow the user to see two different sets of
columns at the same time).

Also, we plan to make the drawer behaviour cease when a different tab is
selected.

Finally, one variable that we use, mfPendingTabBarWidth, is used elsewhere and
we need to ensure that we aren't breaking whatever else uses it.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-02-17 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #13 from Forest Johnson johns...@beloit.edu 2012-02-17 10:55:46 
PST ---
Thanks for the help, we finally got a response on IRC that suggested using some 
other linux distribution might help. We switched to Fedora and now it is
working again.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-02-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #12 from Michael Meeks michael.me...@novell.com 2012-02-08 
06:43:42 PST ---
 I'm sorry that this is off topic, but this is the only place we have been
 able to get in contact with knowledgeable developers.

We are all (and far more of us) on the developers list. You need to ask small,
nice, self contained questions there with all the relevant details.

 After we got back from winter break, our build environments were changed,
 and were not backed up, and now we are having build issues that we
 can't resolve.

That is annoying of course- but without a reasonably complete build log, and
some more interactive help [ such as that you can get on IRC - ask on
#libreoffice-dev on irc.freenode.net ], it is hard to help here.

 Here are the key bits of information we need. All I'm asking for is
 a trivial response.

Quite possibly the problem is not trivial.

 - Has anyone else been having build problems recently in smoketestoo_native
 with a missing cliureversion.mk? If you want details, here is a post by a 
fellow student about the problem:
 http://lists.freedesktop.org/archives/libreoffice/2012-January/024798.html

Not personally, no.

 - Assuming that I'm starting from a mint fresh ubuntu 10.04 instalation,
 is this the correct process for building an older revision of LibreOffice?
 http://pastebin.com/BVRdPAa2

Why start from Nov 30's of last year:

 git branch -f rollback ec894c5fb90bdee1c56d113d14b0bf524df9f # create
 new branch named rollback starting from the commit indicated by the hash

Looks odd.

 - What is the most popular / best environment in which to build LibreOffice?

As long as it's not windows, it should be reasonably do-able from any Linux; I
use openSUSE of course, but YMMV.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


Re: [Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2012-01-31 Thread Yifan Jiang
Hi Forest,

On Tue, Jan 31, 2012 at 05:47:18PM +, bugzilla-dae...@freedesktop.org wrote:
 https://bugs.freedesktop.org/show_bug.cgi?id=42100
 
 --- Comment #11 from Forest Johnson johns...@beloit.edu 2012-01-31 09:47:18 
 PST ---
 I'm sorry that this is off topic, but this is the only place we have been able
 to get in contact with knowledgeable developers.

To get more help, the dev mailing list is a better place to go :)

libreoff...@lists.freedesktop.org

People actually read the mailing list a lot more than a particular bug.

 Here are the key bits of information we need. All I'm asking for is a trivial
 response.
 
 - Has anyone else been having build problems recently in smoketestoo_native
 with a missing cliureversion.mk? 
 If you want details, here is a post by a fellow student about the problem:
 http://lists.freedesktop.org/archives/libreoffice/2012-January/024798.html

For the particular problem, may be it's worthy of trying:

cd loroot/cli_ure;
build;
build --deliver

Normally, the mk file should be in solver/unxlngi6.pro/bin/cliureversion.mk
after delivery. After which continue to make in loroot.

 - Assuming that I'm starting from a mint fresh ubuntu 10.04 instalation, is
 this the correct process for building an older revision of LibreOffice?
 http://pastebin.com/BVRdPAa2

Generally it should be ok, but it depends on which commit was rollbacked. The
build system updated from time to time, so maybe (not sure) you need some
trivial tune-up to make things work with a pretty old code base. Any way it is
the best to try it out and raise up particular error msg when it doesn't work.

 - What is the most popular / best environment in which to build LibreOffice?

It depends on your preference, generally I feel popular Linux distros are
easier since they are used by most developers.

Best wishes,
Yifan
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2011-12-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

Björn Michaelsen bjoern.michael...@canonical.com changed:

   What|Removed |Added

  Status Whiteboard|CONFIRMED EasyHack  |EasyHack
   |DifficultyInteresting   |DifficultyInteresting
   |SkillCpp|SkillCpp

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2011-12-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #9 from Forest Johnson johns...@beloit.edu 2011-12-04 18:37:02 
PST ---
I noticed that there are 5 versions of tabview.cxx, I'm assuming that they are
different implementations of the tabview, probably from days past?

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2011-12-04 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #10 from Kohei Yoshida kohei.yosh...@gmail.com 2011-12-04 
18:41:12 PST ---
(In reply to comment #9)
 I noticed that there are 5 versions of tabview.cxx, I'm assuming that they are
 different implementations of the tabview, probably from days past?

No, they each implement different set of ScTabView methods; they are not
different implementations of ScTabView.  This sort of division of source files
is common in this code base.  It was introduced back when C++ compilers has
limit on the size of a source file it could handle (or so I'm told).

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2011-12-02 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #8 from Kohei Yoshida kohei.yosh...@gmail.com 2011-12-02 07:08:50 
UTC ---
(In reply to comment #7)
 Forest - wonderful to have you involved; thanks ! :-)
 
 4. sounds reasonable enough with the '+' sign widget we have in there; but
 Kohei / Eike might want to

Well, I personally don't want to limit user's ability to re-size, since that
would also affect the re-sizability of the horizontal scroll bar (some users
may want it small).

Other than that, all the other points Forest raised sound very good to me. :-)

Welcome to our project, Forest!

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2011-12-01 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

--- Comment #6 from Forest Johnson johns...@beloit.edu 2011-12-01 20:24:55 
PST ---
Myself and some other college students are in a class on open source software
and we are starting to look at this bug as our first real project.

My plan so far is as follows:

1. figure out how to calculate an optimal size for the tab selection part -
either as wide as it needs to be to show all the tabs or a third of the size of
the window, whichever is smaller.

2. figure out an absolute minimum size for the tab part - like where it shows 2
or 3 tabs plus the little scrolly thing.

3. Whenever the window is resized, regardless of whether the user manually set
the size or not, if the size of the tab selection part is less than the
minimum, or more than the maximum, it is clamped. (exception: when the window
is super tiny. At that point no one cares and the current behaviour is fine)

4. limit the users ability to resize the tab part bigger than the size of all
the tabs?

5. update all of this stuff whenever the number of tabs changes, just like on a
resize

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs


[Libreoffice-bugs] [Bug 42100] EasyHack: calc sheet tab widget screen space usage

2011-11-09 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=42100

Björn Michaelsen bjoern.michael...@canonical.com changed:

   What|Removed |Added

Summary|EasyHack: calc sheet tab|EasyHack: calc sheet tab
   |widget ...  |widget screen space usage

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs