Context Navigation

Changes between Initial Version and Version 1 of Ticket #58962, comment 23

initial	v1
7	7	> * `wp_load_options_group()`
8	8
9		I like the idea of separating the names more obviously. Only one suggested change: The second should remain `...options_by_group()` IMO because it's loading options, not an option group. The two functions are conceptually doing the same thing, and the latter is a convenience wrapper. So IMO we shouldn't change the noun/object~~ive~~ of ''what'' is loaded.
	9	I like the idea of separating the names more obviously. Only one suggested change: The second should remain `...options_by_group()` IMO because it's loading options, not an option group. The two functions are conceptually doing the same thing, and the latter is a convenience wrapper. So IMO we shouldn't change the noun/object of ''what'' is loaded.
10	10
11	11	The other potential concern is that existing option functions don't use the `wp_` prefix, but I'm not opposed to introducing it here - also since the option autoload functions from #58964 already went with `wp_` anyway. Just sharing that as a further naming consideration, but I'm fine with having the `wp_` prefix on them.