[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Pan-devel] Space reading
From: |
Charles Kerr |
Subject: |
Re: [Pan-devel] Space reading |
Date: |
Mon, 27 May 2002 18:19:38 -0700 |
User-agent: |
Mutt/1.3.20i |
Since I'm in the middle of spring cleaning Pan and removing little-used
preferences options, I'm hesitant to apply this patch. Maybe you could
bounce it over to pan-users and see how many people will speak up for
this?
cheers,
Charles
On Fri, May 24, 2002 at 04:04:31PM -0400, Sam Solon wrote:
> The current implementation of space reading really drives me crazy. When
> there's a long article I end up space, space, space, space, space, and
> usually overshoot the end of the article right into the next one.
>
> The "n" key is so close to the space bar that there is really very
> little effort involved in hitting it to go to the next article.
>
> Since there are probably lots of people who love the way it currently
> works -- I'll propose an option to have space reading advance to the
> next article (the cowards way out -- an option) with the following
> patch:
>
>
> RCS file: /cvs/gnome/pan/pan/prefs.c,v
> retrieving revision 1.266
> diff -u -c -r1.266 prefs.c
> *** pan/prefs.c 2002/05/19 14:31:44 1.266
> --- pan/prefs.c 2002/05/24 19:58:06
> ***************
> *** 111,116 ****
> --- 111,117 ----
> GtkWidget * fetch_new_on_group_load_cbutton;
> GtkWidget * fetch_new_on_startup_cbutton;
> GtkWidget * fetch_new_and_bodies_on_startup_cbutton;
> + GtkWidget * space_reading_advances_cbutton;
> GtkWidget * break_thread_when_subject_changes_cbutton;
> GtkWidget * expand_all_threads_by_default_cbutton;
> GtkWidget * hide_mpart_child_nodes_cbutton;
> ***************
> *** 221,226 ****
> --- 222,228 ----
> gboolean fetch_new_on_group_load = TRUE;
> gboolean fetch_new_on_startup = FALSE;
> gboolean fetch_new_and_bodies_on_startup = FALSE;
> + gboolean space_reading_advances = FALSE;
> gboolean header_pane_is_threaded = TRUE;
> gboolean single_click_selects = FALSE;
> gboolean show_group_pane = TRUE;
> ***************
> *** 570,575 ****
> --- 572,582 ----
> win->fetch_new_and_bodies_on_startup_cbutton);
>
> update_entry_and_bool_from_toggle_button (
> + &space_reading_advances,
> + "/Pan/General/space_reading_advances",
> + win->space_reading_advances_cbutton);
> +
> + update_entry_and_bool_from_toggle_button (
> &remove_failed_tasks,
> "/Pan/General/remove_failed_tasks",
> win->remove_failed_tasks);
> ***************
> *** 923,928 ****
> --- 930,936 ----
> wrap_column = pan_config_get_int ("wrap_column=74");
> fetch_new_on_group_load = pan_config_get_bool
> ("fetch_new_on_group_load=TRUE");
> fetch_new_and_bodies_on_startup = pan_config_get_bool
> ("fetch_new_and_bodies_from_subscribed_on_startup=FALSE");
> + space_reading_advances = pan_config_get_bool
> ("space_reading_advances=FALSE");
> hide_mpart_child_nodes = pan_config_get_bool
> ("hide_mpart_child_nodes=TRUE");
> break_thread_when_subject_changes = pan_config_get_bool (
> "break_thread_when_subject_changes=FALSE");
> ***************
> *** 1510,1515 ****
> --- 1518,1530 ----
> gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON(w),
> fetch_new_and_bodies_on_startup);
> connect_signal_to_prefs_changed (w, "toggled");
> win->fetch_new_and_bodies_on_startup_cbutton = w;
> + gtk_box_pack_start (GTK_BOX(vbox), w, FALSE, FALSE, 0);
> +
> + /* space reading advances next article */
> + w = gtk_check_button_new_with_label (_("Space reading advances to
> next article"));
> + gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON(w),
> space_reading_advances);
> + connect_signal_to_prefs_changed (w, "toggled");
> + win->space_reading_advances_cbutton = w;
> gtk_box_pack_start (GTK_BOX(vbox), w, FALSE, FALSE, 0);
>
> /* separator */
> Index: pan/prefs.h
> ===================================================================
> RCS file: /cvs/gnome/pan/pan/prefs.h,v
> retrieving revision 1.78
> diff -u -c -r1.78 prefs.h
> *** pan/prefs.h 2002/05/19 14:31:44 1.78
> --- pan/prefs.h 2002/05/24 19:58:06
> ***************
> *** 35,40 ****
> --- 35,41 ----
> extern gint text_window_smooth_scrolling_speed;
> extern gboolean fetch_new_on_startup;
> extern gboolean fetch_new_and_bodies_on_startup;
> + extern gboolean space_reading_advances;
> extern gboolean fetch_new_on_group_load;
> extern gboolean expand_all_threads_by_default;
> extern gboolean hide_mpart_child_nodes;
> Index: pan/text.c
> ===================================================================
> RCS file: /cvs/gnome/pan/pan/text.c,v
> retrieving revision 1.296
> diff -u -c -r1.296 text.c
> *** pan/text.c 2002/05/22 12:15:21 1.296
> --- pan/text.c 2002/05/24 19:58:06
> ***************
> *** 379,385 ****
> else if ((v->upper < v->page_size) || (val == v->value))
> {
> /* or if we're at the end of a page, go to the next page */
> ! articlelist_read_next ();
> }
> else if (Pan.text->parent==scrolled_window &&
> text_window_smooth_scrolling)
> {
> --- 379,386 ----
> else if ((v->upper < v->page_size) || (val == v->value))
> {
> /* or if we're at the end of a page, go to the next page */
> ! if (space_reading_advances)
> ! articlelist_read_next ();
> }
> else if (Pan.text->parent==scrolled_window &&
> text_window_smooth_scrolling)
> {
>
>
>
> _______________________________________________
> Pan-devel mailing list
> address@hidden
> http://mail.freesoftware.fsf.org/mailman/listinfo/pan-devel