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.

Closed by  speachy
2020-07-03 03:46
Reason for closing:  Accepted

Removed the ugly boxes..

Removed the ugly boxes..

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

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.

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

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.

Resync (just offset lines)

needs re-syncing, i believe

Oh yes it dose

sync r20712.

Sync to r23221.

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.

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. :-)

Sync to r28551.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing