Issue 86377 - Cell number format commands should be toggle-able much like the bold, italic buttons
Summary: Cell number format commands should be toggle-able much like the bold, italic ...
Status: CONFIRMED
Alias: None
Product: Calc
Classification: Application
Component: formatting (show other issues)
Version: recent-trunk
Hardware: All All
: P3 Trivial (vote)
Target Milestone: ---
Assignee: AOO issues mailing list
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-02-22 21:45 UTC by kyoshida
Modified: 2013-03-11 14:58 UTC (History)
5 users (show)

See Also:
Issue Type: PATCH
Latest Confirmation in: ---
Developer Difficulty: ---


Attachments
proposed patch (9.01 KB, patch)
2008-02-22 21:47 UTC, kyoshida
no flags Details | Diff
update patch (15.10 KB, patch)
2008-08-08 21:36 UTC, kyoshida
no flags Details | Diff
update patch to fix a crash bug (i#92703). (15.10 KB, patch)
2008-08-13 03:49 UTC, kyoshida
no flags Details | Diff
revised patch (15.23 KB, patch)
2008-10-22 02:27 UTC, kyoshida
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this issue.
Description kyoshida 2008-02-22 21:45:46 UTC
There are 6 cell number format toolbar buttons (currency, percent, date,
scientific, time and the standard) that apply their respective cell format when
pressed.  But compared to the other formatting buttons such as the bold, italic,
underline, and the alignment buttons, these number format buttons are not
toggle-able.  The patch I will attach shortly will make them toggle-able to make
them behave consistently with the rest of the cell format buttons.

The patch will also remove the standard format since that's no longer needed.
Comment 1 kyoshida 2008-02-22 21:46:19 UTC
cc'ing fl for usability input.
Comment 2 kyoshida 2008-02-22 21:47:14 UTC
Created attachment 51672 [details]
proposed patch
Comment 3 kyoshida 2008-02-22 21:47:33 UTC
issue type PATCH
Comment 4 niklas.nebel 2008-06-17 15:25:10 UTC
This needs user Experience input, especially removing the "Standard" button.

The handling of multiple cells in the patch (just using the cursor position) is
different from other format buttons.
Comment 5 kyoshida 2008-08-08 21:36:31 UTC
Created attachment 55663 [details]
update patch
Comment 6 kyoshida 2008-08-08 21:37:55 UTC
The new patch handles the state of a number format on a cell range the same way
as how a font attribute state is decided.
Comment 7 kyoshida 2008-08-13 03:49:59 UTC
Created attachment 55730 [details]
update patch to fix a crash bug (i#92703).
Comment 8 adajun 2008-10-21 11:02:59 UTC
The standard format button is needed for reset Add/Delete decimal place.
So it should not be removed.
Comment 9 kyoshida 2008-10-21 17:44:40 UTC
@adajun: right-clicking on a cell and selecting the 'Default Formatting' does
the same thing as the Standard button.  I personally don't think it warrants an
extra icon since the functionality is available from the context menu.
Comment 10 Regina Henschel 2008-10-21 18:43:11 UTC
No, context menu > "Default Formatting" removes hard formatting but the assigned
cell style remains. The "standard format" button sets a hard formatting. You can
easily see the difference if you define a cell style with a science format of 15
digits, for example.

I need the "standard format" button, do not remove it.
Comment 11 adajun 2008-10-22 02:00:09 UTC
@kohei:It is not same, for example "Default Formatting" removes border setting
too. Right-clicking on a cell and select the 'Format cells' and select Numbers
tab and Number in category and General in Format. It is difficult to select the
suitable item in Format cells for beginners.

If it is able to set using toolbar button, then it should be able to unset using
toolbar button. It is very important thing for usability.
Comment 12 kyoshida 2008-10-22 02:27:16 UTC
Created attachment 57370 [details]
revised patch
Comment 13 kyoshida 2008-10-22 02:35:05 UTC
Ok, fair argument.  I've revised my patch to revive the standard button but made
invisible by default.  The button can be easily turned on via toolbar's 'Visible
Buttons' properties.

By the way, 

>If it is able to set using toolbar button, then it should be able to unset
using toolbar button.

this is what my toggle-able format button does.  Setting it will set the
respective cell format, and pressing the same button again will set the cell
format back to the standard.  So, what you said above is already taken into
consideration (hence the argument for removing the standard button).
Comment 14 adajun 2008-10-22 04:32:19 UTC
Thank you for accepting.
Toggle-able format buttons are easy to understand, I think.
But to use Add decimal place button or Delete decimal place button
we needs standard format button.
Two problems are remained.
1. The icon of standard format button is designed for reset
   currency and percent setting. We can see $ and % and arrow in it.
   It has still the function, but
   it does not match to toggle-able button.
2. It same to windows version ? I do not use 3.0 yet.
   OOo2.4.1 for windows has non toggle-able buttons.

Comment 15 kyoshida 2008-10-22 16:24:36 UTC
>1. The icon of standard format button is designed for reset
>   currency and percent setting. We can see $ and % and arrow in it.
>   It has still the function, but
>   it does not match to toggle-able button.

Well, icon design is a separate issue.  Since the standard icon is used like
that today, I don't see the urgent need to change its design.  (IMO the icon
should be symbolic of what it does, but it's hard to be "accurate" with its design.)

>2. It same to windows version ? I do not use 3.0 yet.
>   OOo2.4.1 for windows has non toggle-able buttons.

Ah, you must be new here. :-)  See the status of this issue (NEW), and the
target milestone not being set.  That means this patch has not been accepted
yet.  In other words, this functionality is not available in any versions you
can download from the official site.
Comment 16 adajun 2008-10-23 09:34:15 UTC
>icon design is a separate issue.
I see. I hope someone make it to be accurate too :-)

>you must be new here
Yes, sorry. I realized after send.
Debian's OOo2.4.1 is toggle-able already, so
I misunderstand.
Comment 17 aoo-bugger 2009-01-21 16:49:15 UTC
@kohei
maybe you can take a look at issue
[[http://qa.openoffice.org/issues/show_bug.cgi?id=8443|8443]], as it is also
about a toggle-able enhancement.
Thanks.
Comment 18 Rob Weir 2013-03-11 14:58:37 UTC
I'm adding this comment to all open issues with Issue Type == PATCH.  We have 220 such issues, many of them quite old.  I apologize for that.  

We need your help in prioritizing which patches should be integrated into our next release, Apache OpenOffice 4.0.

If you have submitted a patch and think it is applicable for AOO 4.0, please respond with a comment to let us know.

On the other hand, if the patch is no longer relevant, please let us know that as well.

If you have any general questions or want to discuss this further, please send a note to our dev mailing list:  dev@openoffice.apache.org

Thanks!

-Rob