Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Plugins
  • Assigned To No-one
  • Operating System iPod 5G
  • Severity Low
  • Priority Very Low
  • Reported Version Daily build (which?)
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by yhuang - 2008-11-27
Last edited by speachy - 2020-07-03

FS#9583 - periodic table

I found this app useful in iPodLinux so I decided to port it to Rockbox. Jonathan Bettencourt is the original author.

Works on daily build: 19242 for iPod Video

Smaller screen targets won’t be able to see the full table.

   periodic.diff (17.6 KiB)
   dump 081127-165027.bmp (150.1 KiB)
Closed by  speachy
2020-07-03 03:46
Reason for closing:  Accepted
yhuang commented on 2008-11-27 23:54

Removed the ugly boxes..

   periodic.diff (18.1 KiB)
   dump 081127-184130.bmp (150.1 KiB)
yhuang commented on 2008-11-28 00:45

Removed the ugly boxes..

   periodic.diff (18.1 KiB)
   dump 081127-184130.bmp (150.1 KiB)
scotty commented on 2008-12-01 12:22

Replaced strlen() calls by rb→strlen() to make it compile for ARM targets.
Fixed 2 warnings ("function declaration isn’t a prototype").

   periodic.diff (18.1 KiB)
dionoea commented on 2008-12-07 16:25

Current policy is to only accept plugins that work on all targets (or on color targets for those that obviously couldn't work on black and white screens). Would it be possible to make the screen scroll so it could work on smaller screens? Thanks.

ltlollo commented on 2009-01-19 13:21

If compiled for Ipod Video, on 3.1 sources, some informations on the table are missing; on uisimulator the infos are present.

   dump 090119-141614.bmp (150.1 KiB)
   UISIMdump 090119-141917.bmp (150.1 KiB)
Uchida commented on 2009-01-27 09:22
Warning: Undefined array key "useheading" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parser/xhtml.php on line 1099 Warning: Undefined array key "target" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parser/xhtml.php on line 557 Warning: Trying to access array offset on value of type null in /home/rockbox/flyspray/plugins/dokuwiki/inc/parser/xhtml.php on line 557

In periodic plugin, there is the following problems:
- Build of this plugin fail under the current version Rockbox.
- It does not customize for players other than iPod video.
- The source code doesn't follow the Rockbox coding rule. ("" comment, using TAB)
- Physical data of elements (mass, dencity, freezing point, and boiling point) is old.
- I do not understand whether it is uncertain data or the value of data is 0 because uncertain data is output by 0. I corrected the source of this plugin in place of Huang.
The problem of lorenzo is solved by this correction. changes:
- Build of this plugin success under the current version Rockbox.
- Current Rockbox build player (http://build.rockbox.org/) supports (excepts Archos Player).
(I check actual iPod video. Other players checked by using uisimulator.
- TAB → 4 spaces, coments → /* comments */
- Physical data of elements update.
- Physical data changes float to char *.
- Uncertain data outputs by "-".
- Element has no stable nuclides. The value enclosed in brackets. (except Th, Pa, and U)
- Lantanoids include "La", Actinoids include "Ac" (tables place move)
- The 117th element does not display.

   periodic_new.patch (31.9 KiB)
asettico commented on 2009-03-03 14:59

Resync (just offset lines)

   plugins-9583periodic_new-v6.1... (31.9 KiB)
froggyman commented on 2009-04-11 00:59

needs re-syncing, i believe

Thecoolgman commented on 2009-04-15 00:51

Oh yes it dose

Uchida commented on 2009-04-16 13:30

sync r20712.

   periodic_new_2.patch (31.9 KiB)
asettico commented on 2009-10-17 09:26

Sync to r23221.

   plugins-9583periodic_new-v6.1... (31.9 KiB)
Ubuntuxer commented on 2009-10-18 10:21

In my opionion a informative periodic table is very useful, but there is still some work, before it can be committed.

The plugin have to work on stable ports as well as unstable ports, so I add some button macros. Meanwhile the plugin doesn't work on the c200, because of the small display size. I can't figure out, how to solve this.
But also on the e200 the font is almost unreadable. Perhaps it should be turned, because on the fuze it looks pretty good, although just high and width size is swapped.

Besides the application needs an manual.
For what is the button PERIODIC_KEY_SELECT ?

@asettico: Please use the the command "svn diff apps/plugins/ > patch.diff" to create a patch.

   periodic_new_3.diff (33.3 KiB)
asettico commented on 2009-10-18 10:44

Normally I use the command "svn diff", but in this case I fixed the patch more quickly simply modifying it, so the git format was unchanged. :-)

scotty commented on 2010-11-11 00:07

Sync to r28551.

   FS9583-periodic_new_3-r28551.... (33.2 KiB)

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing