changed make active to target data-id