todo inline totally functional
close edit works