Bug 6035 - Podcast settings page behavior confusing
: Podcast settings page behavior confusing
Status: CLOSED FIXED
Product: Logitech Media Server
Classification: Unclassified
Component: Web Interface
: 7.0
: Macintosh MacOS X 10.4
: P2 normal (vote)
: 7.x
Assigned To: Michael Herger
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2007-11-04 05:29 UTC by David Myers
Modified: 2009-07-31 10:15 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments
set value and submit via JS (1.67 KB, patch)
2007-11-04 18:11 UTC, KDF
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Myers 2007-11-04 05:29:03 UTC
The podcast settings page is a bit confusing.  It's not immediately obvious that you must NOT hit <RETURN> after typing in a podcast URL.  Instead you must press Save Settings, which is not near the text entry field on the page.

If you hit <RETURN> after entering a podcast URL, the new podcast is added, all other podcasts are deleted, and the defaults return.

SC7 nightly 11/4 on Fedora 7
Firefox 2.0.0.9 on Mac OS 10.4.10
Comment 1 KDF 2007-11-04 16:15:41 UTC
RETURN submits the form, which means the reset button is triggered.
Same problem likely exists for RSS News plugin
Comment 2 KDF 2007-11-04 18:11:30 UTC
Created attachment 2352 [details]
set value and submit via JS

This is one way around the problem.  The reset button doesn't submit on return, only on click.  the resetpodcasts param is then able to be used to trigger the reset to defaults.
Comment 3 Michael Herger 2007-11-08 07:43:51 UTC
Did you see this in all skins or just Default? 

IMHO we're already covering this case for all skins but Default (see settings.js -> initSettingsForm()). Must do for Default, too.
Comment 4 David Myers 2007-11-09 04:23:13 UTC
I was using Default.
Comment 5 Michael Herger 2007-11-12 06:47:57 UTC
Change 14634 - on Enter/Return key don't launch the first submit button in the form, but always the "Save settings" button.

Please test and re-open if needed.
Comment 6 David Myers 2007-11-13 04:35:18 UTC
Looks good.

Thanks.
Comment 7 Chris Owens 2008-07-30 15:30:05 UTC
This bug has now been fixed in the 7.1 release version of SqueezeCenter!  Please download the new version from http://www.slimdevices.com if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.
Comment 8 Chris Owens 2009-07-31 10:15:14 UTC
Reduce number of active targets for SC